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

Create linter.yml #440

Closed
wants to merge 11 commits into from
Closed

Create linter.yml #440

wants to merge 11 commits into from

Conversation

bbannon
Copy link
Contributor

@bbannon bbannon commented Nov 25, 2023

Let github lint everything.

Let github lint everything.
Super linter said lines too long.
Need the folder for the reports.
We will need to set up the rules for each language.
@bbannon
Copy link
Contributor Author

bbannon commented Nov 25, 2023

Well damn. Need the folder, but not the file. I don't see how from here.

#############################
on:
push:
branches-ignore: [master, main]
Copy link
Owner

Choose a reason for hiding this comment

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

Would be good to include "release" branch as well, but I can add that later

@craigk5n
Copy link
Owner

Can you explain what this is supposed to do? When I look at the action output, it doesn't look like it scanned any of the webcalendar files:

https://github.com/craigk5n/webcalendar/actions/runs/7045472464/job/19175266697

@bbannon
Copy link
Contributor Author

bbannon commented Dec 2, 2023 via email

@bbannon
Copy link
Contributor Author

bbannon commented Dec 2, 2023

I'll reopen this when I get it to work.

@bbannon bbannon closed this Dec 2, 2023
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