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

Fix: CI Workflow #41

Merged
merged 2 commits into from
May 6, 2024
Merged

Fix: CI Workflow #41

merged 2 commits into from
May 6, 2024

Conversation

Drish-xD
Copy link
Collaborator

@Drish-xD Drish-xD commented Apr 13, 2024

#38 Fixed the version of pre-commit/action from v3.0.1 -> v2.0.3

@Drish-xD
Copy link
Collaborator Author

Hey @suryabulusu, Please review the PR.

@Drish-xD Drish-xD changed the title FIX CI Workflow Fix: CI Workflow Apr 13, 2024
Copy link
Contributor

@suryabulusu suryabulusu left a comment

Choose a reason for hiding this comment

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

hey thanks! can you add DMP 2024 and other things in .gitignore? I just approved pre-commit github actions run for this PR and it seems to fail because of issue template. also please do submit a proposal! @Drish-xD

@Drish-xD
Copy link
Collaborator Author

Okay, will do it by today.

And I submitted the proposal yesterday. 🚀🚀✌️

@Drish-xD
Copy link
Collaborator Author

@suryabulusu I have fixed the CI Issue by excluding the Template folder in the .github/ dir from trailing-whitespace hook.
image

Copy link
Contributor

@suryabulusu suryabulusu left a comment

Choose a reason for hiding this comment

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

looks good. thanks!

@suryabulusu suryabulusu merged commit 2d307be into avantifellows:main May 6, 2024
1 check 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.

2 participants