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

chore: add script to update core dependencies #597

Merged

Conversation

dyladan
Copy link
Member

@dyladan dyladan commented Jul 28, 2021

Provide a script for automating the core dependency update process. This only bumps the versions and the person who runs it should also make any required code changes.

@dyladan dyladan requested a review from a team as a code owner July 28, 2021 20:15
@codecov
Copy link

codecov bot commented Jul 28, 2021

Codecov Report

Merging #597 (b5d9b93) into main (2d41152) will not change coverage.
The diff coverage is n/a.

❗ Current head b5d9b93 differs from pull request most recent head 2fc47e5. Consider uploading reports for the commit 2fc47e5 to get more accurate results

@@           Coverage Diff           @@
##             main     #597   +/-   ##
=======================================
  Coverage   94.94%   94.94%           
=======================================
  Files         195      195           
  Lines       11637    11637           
  Branches     1115     1115           
=======================================
  Hits        11049    11049           
  Misses        588      588           

@dyladan dyladan force-pushed the update-core-script branch 2 times, most recently from dc16a0c to a4ee738 Compare July 28, 2021 20:30
scripts/update-core-deps.js Outdated Show resolved Hide resolved
@obecny
Copy link
Member

obecny commented Jul 29, 2021

can you add in the header of the file explanation how to use it and if we should be aware of something (to do or not to do) ? can this accept a param with either a version or path to the opentelemetry-js ?

@vmarchaud
Copy link
Member

Will need a rebase @dyladan

@dyladan dyladan merged commit 95aa84f into open-telemetry:main Aug 9, 2021
@dyladan dyladan deleted the update-core-script branch August 9, 2021 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants