Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Add blocked protective stop #1947

Merged
merged 1 commit into from
Jan 15, 2025
Merged

Add blocked protective stop #1947

merged 1 commit into from
Jan 15, 2025

Conversation

oysand
Copy link
Contributor

@oysand oysand commented Jan 15, 2025 •
edited
Loading

Ready for review checklist:

  • A self-review has been performed
  • All commits run individually
  • Temporary changes have been removed, like console.log, TODO, etc.
  • The PR has been tested locally
  • A test have been written
    • This change doesn't need a new test
  • Relevant issues are linked
  • Remaining work is documented in issues
    • There is no remaining work from this PR that require new issues
  • The changes does not introduce dead code as unused imports, functions etc.

@github-actions GitHub Actions
Copy link

🔔 Changes in database folder detected 🔔
Do these changes require adding new migrations? 🤔 In that case follow these steps.
If you are uncertain, ask a database admin on the team 😄

@oysand oysand force-pushed the add-blocked-protective-stop branch from ac5fb56 to 9986b5e Compare January 15, 2025 10:17
Christdej
Christdej approved these changes Jan 15, 2025
View reviewed changes
Copy link
Contributor

@Christdej Christdej left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@oysand oysand merged commit 4191785 into equinor:main Jan 15, 2025
13 checks passed
@oysand oysand deleted the add-blocked-protective-stop branch January 15, 2025 11:10
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@Christdej Christdej Christdej approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@oysand @Christdej