Skip to content

Log warning for tasks outside area polygon#2059

Merged
Christdej merged 1 commit intoequinor:mainfrom
Christdej:logpolygon
Mar 5, 2025
Merged

Log warning for tasks outside area polygon#2059
Christdej merged 1 commit intoequinor:mainfrom
Christdej:logpolygon

Conversation

@Christdej
Copy link
Copy Markdown
Contributor

@Christdej Christdej commented Mar 5, 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.

@Christdej Christdej added the improvement Improvement to existing functionality label Mar 5, 2025
@Christdej Christdej self-assigned this Mar 5, 2025
andchiind
andchiind approved these changes Mar 5, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@andchiind andchiind left a comment

Choose a reason for hiding this comment

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

Very nice, good work

@Christdej Christdej merged commit 26c5f12 into equinor:main Mar 5, 2025
12 checks passed
@Christdej Christdej deleted the logpolygon branch March 5, 2025 06:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@andchiind andchiind andchiind approved these changes

Assignees

@Christdej Christdej

Labels

improvement Improvement to existing functionality

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Christdej @andchiind