Skip to content

bump datadog-ci dependency to 0.2#3186

Merged
anmarchenko merged 1 commit into
masterfrom
anmarchenko/ci_vis_0.2
Oct 5, 2023
Merged

bump datadog-ci dependency to 0.2#3186
anmarchenko merged 1 commit into
masterfrom
anmarchenko/ci_vis_0.2

Conversation

@anmarchenko

@anmarchenko anmarchenko commented Oct 5, 2023

Copy link
Copy Markdown
Member

What does this PR do?
Bumps datadog-ci runtime dependency to 0.2

Release notes:

Added

  • Agentless mode

Fixed

  • Fix an issue with emojis in commit message breaking LocalGit tags provider

Motivation:
Introducing agentless mode for CI visibility product

How to test the change?

git clone https://github.com/DataDog/test-environment

cd test-environment

yarn

cd dd-trace-rb/jekyll
bundle add ddtrace --git "https://github.com/DataDog/dd-trace-rb.git" --branch "anmarchenko/ci_vis_0.2" --skip-install
bundle install --path .bundle
cd ../..

yarn test dd-trace-rb/jekyll.test.js

cd dd-trace-rb/devdocs
bundle add ddtrace --git "https://github.com/DataDog/dd-trace-rb.git" --branch "anmarchenko/ci_vis_0.2" --skip-install
bundle install --path .bundle
cd ../..

yarn test dd-trace-rb/devdocs.test.js

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.

@anmarchenko
anmarchenko requested a review from a team as a code owner October 5, 2023 10:41
ivoanjo
ivoanjo approved these changes Oct 5, 2023
View reviewed changes

@ivoanjo ivoanjo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍 LGTM

@anmarchenko
anmarchenko force-pushed the anmarchenko/ci_vis_0.2 branch from c323c09 to 825473d Compare October 5, 2023 11:27
@anmarchenko
anmarchenko force-pushed the anmarchenko/ci_vis_0.2 branch from 825473d to 5d70a43 Compare October 5, 2023 14:50
@anmarchenko
anmarchenko merged commit 1ee7bcf into master Oct 5, 2023
@anmarchenko
anmarchenko deleted the anmarchenko/ci_vis_0.2 branch October 5, 2023 15:09
@github-actions github-actions Bot added this to the 1.15.0 milestone Oct 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ekump ekump ekump approved these changes
@ivoanjo ivoanjo ivoanjo approved these changes
+1 more reviewer
@GustavoCaso GustavoCaso GustavoCaso approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

1.15.0

Development

Successfully merging this pull request may close these issues.

4 participants

@anmarchenko @ekump @ivoanjo @GustavoCaso