Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

[MAINTENANCE] Pytest Mark - ExecutionEngine #8324

Merged
merged 19 commits into from Jul 14, 2023
Merged

[MAINTENANCE] Pytest Mark - ExecutionEngine #8324

merged 19 commits into from Jul 14, 2023

Conversation

Shinnnyshinshin
Copy link
Contributor

@Shinnnyshinshin Shinnnyshinshin commented Jul 14, 2023 •
edited

  • Pytest Mark for tests/execution_engine folder

  • Description of PR changes above includes a link to an existing GitHub issue

  • PR title is prefixed with one of: [BUGFIX], [FEATURE], [DOCS], [MAINTENANCE], [CONTRIB]

  • Code is linted - run invoke lint (uses black + ruff)

  • Appropriate tests and docs have been updated

@netlify
Copy link

netlify bot commented Jul 14, 2023 •
edited

✅ Deploy Preview for niobium-lead-7998 canceled.

Name Link
🔨 Latest commit 87a8722
🔍 Latest deploy log https://app.netlify.com/sites/niobium-lead-7998/deploys/64b1a85e6d5a820009f881c5

@Shinnnyshinshin Shinnnyshinshin changed the base branch from develop to m/pp-232/data-context-folders July 14, 2023 19:50
@Shinnnyshinshin Shinnnyshinshin self-assigned this Jul 14, 2023
Base automatically changed from m/pp-232/data-context-folders to develop July 14, 2023 19:53
@Shinnnyshinshin Shinnnyshinshin enabled auto-merge (squash) July 14, 2023 19:56
@ghost
Copy link

ghost commented Jul 14, 2023 •
edited by ghost

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

@Shinnnyshinshin Shinnnyshinshin merged commit 6ebc569 into develop Jul 14, 2023
55 checks passed
@Shinnnyshinshin Shinnnyshinshin deleted the m/pp-232/execution-engine-and-docs branch July 14, 2023 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Kilo59 Kilo59 Kilo59 approved these changes

Assignees

@Shinnnyshinshin Shinnnyshinshin

Labels
core dx
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Shinnnyshinshin @Kilo59