-
-
Notifications
You must be signed in to change notification settings - Fork 566
feat: Docker Pre-Commit-Hooks #925
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
base: master
Are you sure you want to change the base?
Changes from all commits
d8bb39a
f326080
d70be2c
3b18d08
1b78031
3e91a9c
bb38650
17aa365
aa0d873
8c61c89
0f8dfaa
6025c14
c40f434
c699975
d97a860
2f2bca6
f26e742
4400849
df1fb32
db8b255
0751a04
506466c
75a3a51
f35d066
12e5de2
8455c2e
1731092
8fb27aa
2f4a2a3
5aab062
dd977d1
2dec291
70a37a8
d1629ba
4050d10
1815705
5aed527
75e7837
3d20103
9e0a309
246cbe5
1108a8f
d848eb1
6b2eb62
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,8 @@ | ||
* | ||
!.dockerignore | ||
!Dockerfile | ||
!hooks/*.sh | ||
!lib_getopt | ||
!src/ | ||
!tools/entrypoint.sh | ||
!tools/install/*.sh |
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. largely the same, but got rid of the need for your if schedule do these jobs if not do these other jobs by using one of the other standard docker actions. was able to clean up the jobs a bit and basically do the same thing - you might want to double check though |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -14,6 +14,7 @@ name: CodeQL | |
on: | ||
push: | ||
branches: | ||
- main | ||
- master | ||
merge_group: | ||
pull_request: | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. added main because i normally fork to master and only allow upstream updates to come in, then use main for my main in case i want to use it before a PR is approved. Also made it both docker files and the release workflow so i could trigger it when i changed it. modified to use built-in github_token because that is the only way for "fresh/new" packages to auto attribute to parent repo, PAT orphans them until you go add them to a repo. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Got it. Now I'll modify them back |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -13,6 +13,7 @@ on: | |
- cron: 20 7 * * 2 | ||
push: | ||
branches: | ||
- main | ||
- master | ||
|
||
# Declare default permissions as read only. | ||
|
Uh oh!
There was an error while loading. Please reload this page.