Skip to content

NO-SNOW add flag for local test setup #2397

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.

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

Merged
merged 3 commits into from
Jul 9, 2025

Conversation

sfc-gh-pczajka
Copy link
Contributor

@sfc-gh-pczajka sfc-gh-pczajka commented Jul 9, 2025

Fill out the following pre-review checklist:

  • I am adding a new automated test(s) to verify correctness of my new code
  • I am adding new logging messages
  • I am adding a new telemetry message
  • I am modifying authorization mechanisms
  • I am adding new credentials
  • I am modifying OCSP code
  • I am adding a new dependency

Please describe how your code solves the related issue.

The code now runs in the same way on the local environment as it was running on GH Actions (detected as "is_public_test"). This PR adds "is_local_dev_setup" fixture, which can be marked in parameters.py, and adds it in tests where "is_public_test" fixture is used

@sfc-gh-pczajka sfc-gh-pczajka added NO-CHANGELOG-UPDATES This pull request does not need to update CHANGELOG.md DO_NOT_PORT_CHANGES_TO_SP Add this label when changes in this PR do not need to be port to SP connector labels Jul 9, 2025
@sfc-gh-pczajka sfc-gh-pczajka enabled auto-merge (squash) July 9, 2025 12:40
@sfc-gh-pczajka sfc-gh-pczajka merged commit 456cb39 into main Jul 9, 2025
98 checks passed
@sfc-gh-pczajka sfc-gh-pczajka deleted the NO-SNOW-add-flag-for-local-test-setup branch July 9, 2025 12:44
@github-actions github-actions bot locked and limited conversation to collaborators Jul 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
DO_NOT_PORT_CHANGES_TO_SP Add this label when changes in this PR do not need to be port to SP connector NO-CHANGELOG-UPDATES This pull request does not need to update CHANGELOG.md
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants