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

Update standard workflow files. #2867

Merged
merged 1 commit into from
Nov 27, 2023

Conversation

feanil
Copy link
Contributor

@feanil feanil commented Nov 27, 2023

This PR was created automatically by the repo_checks tool.

The .github/workflows/commitlint.yml workflow is missing or needs an update to stay in
sync with the current standard for this workflow as defined in the
`.github` repo of the `openedx` GitHub org.
Copy link

netlify bot commented Nov 27, 2023

Deploy Preview for paragon-openedx ready!

Name Link
🔨 Latest commit b50890c
🔍 Latest deploy log https://app.netlify.com/sites/paragon-openedx/deploys/6564af4df52083000889cc2c
😎 Deploy Preview https://deploy-preview-2867--paragon-openedx.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

codecov bot commented Nov 27, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (8d9630b) 92.83% compared to head (b50890c) 92.83%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2867   +/-   ##
=======================================
  Coverage   92.83%   92.83%           
=======================================
  Files         235      235           
  Lines        4240     4240           
  Branches     1029     1029           
=======================================
  Hits         3936     3936           
  Misses        300      300           
  Partials        4        4           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@feanil feanil requested review from PKulkoRaccoonGang and a team November 27, 2023 15:20

- name: Run commitlint
uses: wagoid/commitlint-github-action@v5
uses: openedx/.github/.github/workflows/commitlint.yml@master
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is fixed in the latest version of the commit linter so the override is no longer needed.

Comment on lines -19 to -21
# This step is needed to fix https://github.com/conventional-changelog/commitlint/issues/3256
- name: Remove tsconfig file
run: rm tsconfig.json
Copy link
Contributor

Choose a reason for hiding this comment

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

https://github.com/openedx/.github/blob/master/.github/workflows/commitlint.yml does not include this step.

The issue mentioned here (conventional-changelog/commitlint#3256) is still open, but I tested the repro steps (commitlint --from HEAD~2 --to HEAD) locally and found no errors.

The environment mentioned in the linked issue was:

commitlint --version 16.3.0
git --version 2.36.1
node --version 18.4.0

After running nvm use, my local environment was

commitlint --version 18.4.3
git --version 2.42.0
node --version 18.18.2

The commitlint gh action uses commitlint version ^18.2.0, and node version 20.9.0, so overall this feels safe to remove.

@brian-smith-tcril brian-smith-tcril merged commit 71405c1 into master Nov 27, 2023
10 checks passed
@brian-smith-tcril brian-smith-tcril deleted the repo_checks/ensure_workflows branch November 27, 2023 15:44
@openedx-semantic-release-bot

🎉 This PR is included in version 21.8.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@openedx-semantic-release-bot

🎉 This PR is included in version 22.0.0-alpha.19 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants