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

Ensure PR number is referenced in .unreleased files #5740

Merged
merged 1 commit into from
Jun 2, 2023

Conversation

kgyrtkirk
Copy link
Contributor

@kgyrtkirk kgyrtkirk commented Jun 1, 2023

Adds a simple check to ensure that the PR number is present at least once in the added changelog file.
Also fixes an earlier PR which introduced a typo.

Disable-check: force-changelog-file

@codecov
Copy link

codecov bot commented Jun 1, 2023

Codecov Report

Merging #5740 (a975693) into main (ecc1b7b) will decrease coverage by 0.07%.
The diff coverage is n/a.

❗ Current head a975693 differs from pull request most recent head 28bb343. Consider uploading reports for the commit 28bb343 to get more accurate results

@@            Coverage Diff             @@
##             main    #5740      +/-   ##
==========================================
- Coverage   87.85%   87.79%   -0.07%     
==========================================
  Files         234      234              
  Lines       54987    54982       -5     
  Branches    12114    12113       -1     
==========================================
- Hits        48310    48271      -39     
- Misses       4852     4856       +4     
- Partials     1825     1855      +30     

see 20 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@kgyrtkirk kgyrtkirk force-pushed the pr_num_check branch 2 times, most recently from 2a174e3 to 35edd35 Compare June 1, 2023 13:39
@kgyrtkirk kgyrtkirk marked this pull request as ready for review June 2, 2023 07:51
@github-actions
Copy link

github-actions bot commented Jun 2, 2023

@konskov, @gayyappan: please review this pull request.

Powered by pull-review

Adds a simple check to ensure that the PR number is present at least once
in the added changelog file.
Also fixes an earlier PR which introduced a typo.
@kgyrtkirk kgyrtkirk enabled auto-merge (rebase) June 2, 2023 16:33
@kgyrtkirk kgyrtkirk merged commit ac70906 into timescale:main Jun 2, 2023
55 of 57 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants