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

Fixed the naming of the Windows GitHub action #5791

Merged
merged 1 commit into from
Jun 15, 2023

Conversation

jnidzwetzki
Copy link
Contributor

The ignored workflow for windows-build-and-test does not set the name of the actions properly. Therefore, these actions use the default naming. For example, 'Regression Windows / build (15, windows-2022, Debug)'. However, our CI expects names like 'PG15 Debug windows-2022' in the required checks. This PR corrects the name of the jobs.

Link to a PR with a missing check: #5777

Disable-check: force-changelog-file

The ignored workflow for windows-build-and-test does not set the name of
the actions properly. Therefore, these actions use the default naming.
For example, 'Regression Windows / build (15, windows-2022, Debug)'.
However, our CI expects names like 'PG15 Debug windows-2022' in the
required checks. This PR corrects the name of the jobs.
@jnidzwetzki jnidzwetzki marked this pull request as ready for review June 15, 2023 10:53
@github-actions
Copy link

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

Powered by pull-review

@codecov
Copy link

codecov bot commented Jun 15, 2023

Codecov Report

Merging #5791 (4299a44) into main (4273a27) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #5791      +/-   ##
==========================================
+ Coverage   87.85%   87.87%   +0.01%     
==========================================
  Files         239      239              
  Lines       55483    55478       -5     
  Branches    12288    12287       -1     
==========================================
+ Hits        48747    48750       +3     
+ Misses       4861     4828      -33     
- Partials     1875     1900      +25     

see 26 files with indirect coverage changes

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

@jnidzwetzki jnidzwetzki enabled auto-merge (rebase) June 15, 2023 11:57
@jnidzwetzki jnidzwetzki merged commit f05b7f8 into timescale:main Jun 15, 2023
44 checks passed
@jnidzwetzki jnidzwetzki added the force-auto-backport Automatically backport this PR or fix of this issue, even if it's not marked as "bug" label Jun 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backported-2.11.x force-auto-backport Automatically backport this PR or fix of this issue, even if it's not marked as "bug"
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants