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.

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

Fix Datadog::CI::Environment to support the new CI specs #3080

Merged
merged 2 commits into from
Aug 26, 2023

Conversation

anmarchenko
Copy link
Contributor

@anmarchenko anmarchenko commented Aug 25, 2023

Jira issue

What does this PR do?
Fixes Datadog::CI::Environment to support the new CI specs

Motivation:
Specification for CI environment integration changed and bitbucket repository URL extraction logic must be updated to make datadog-ci-spec tests pass

How to test the change?

Tested via environment_spec.rb test with new fixtures

For Datadog employees:

  • If this PR touches code that signs or publishes builds or packages, or handles
    credentials of any kind, I've requested a review from @DataDog/security-design-and-guidance.
  • This PR doesn't touch any of that.

Unsure? Have a question? Request a review!

@github-actions github-actions bot added the dev/testing Involves testing processes (e.g. RSpec) label Aug 25, 2023
@anmarchenko anmarchenko requested a review from a team August 25, 2023 14:56
@anmarchenko anmarchenko marked this pull request as ready for review August 25, 2023 14:57
@anmarchenko anmarchenko requested a review from a team August 25, 2023 14:57
@github-actions github-actions bot added ci-app CI product for test suite instrumentation and removed dev/testing Involves testing processes (e.g. RSpec) labels Aug 25, 2023
@anmarchenko anmarchenko merged commit 8d59aa2 into master Aug 26, 2023
204 checks passed
@anmarchenko anmarchenko deleted the anmarchenko/fix_ci_specs_for_test_visibility branch August 26, 2023 20:01
@github-actions github-actions bot added this to the 1.15.0 milestone Aug 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-app CI product for test suite instrumentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants