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

Workflow lint readme and prereq tags #433

Merged
merged 11 commits into from
Sep 14, 2023
Merged

Workflow lint readme and prereq tags #433

merged 11 commits into from
Sep 14, 2023

Conversation

uk-bolly
Copy link
Member

Overall Review of Changes:
Added new workflow
lints updates and aligned
readme updated
pre-commit enabled and ready
tags on prereqs added

Issue Fixes:
tags on prereqs thanks to @trae on discord community

Enhancements:
Please list any enhancements/features that are not open issue tickets

How has this been tested?:
Manually, high disruption with audit

Signed-off-by: Mark Bolwell <[email protected]>
Signed-off-by: Mark Bolwell <[email protected]>
Signed-off-by: Mark Bolwell <[email protected]>
Signed-off-by: Mark Bolwell <[email protected]>
Signed-off-by: Mark Bolwell <[email protected]>
Signed-off-by: Mark Bolwell <[email protected]>
Signed-off-by: Mark Bolwell <[email protected]>
Signed-off-by: Mark Bolwell <[email protected]>
Signed-off-by: Mark Bolwell <[email protected]>
@uk-bolly
Copy link
Member Author

workflow will fail as files missing due to tidy up

@uk-bolly uk-bolly requested a review from MrSteve81 September 14, 2023 15:39
Copy link
Contributor

@MrSteve81 MrSteve81 left a comment

Choose a reason for hiding this comment

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

Approved

@uk-bolly uk-bolly merged commit baa1d83 into devel Sep 14, 2023
2 of 3 checks passed
@uk-bolly uk-bolly deleted the workflow_lint_readme branch September 14, 2023 15:46
@uk-bolly uk-bolly mentioned this pull request Sep 14, 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