-
Notifications
You must be signed in to change notification settings - Fork 5
chore: add CI step for validating contributor READMEs #1
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
Merged
Merged
Changes from all commits
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
96c20c4
chore: add sample README
Parkreiner a5c4495
wip: commit progress on CI validation script
Parkreiner 20204b0
chore: add extra sample data file
Parkreiner 902b32f
wip: commit more progress on script
Parkreiner 1906520
chore: add logs for better feedback
Parkreiner da735da
fix: remove parsing bugs
Parkreiner b191321
fix: make logging better
Parkreiner 629f4b3
chore: rename directory for script
Parkreiner 2b9da92
refactor: remove goto statements
Parkreiner 23f1cee
fix: remove accidental segfault
Parkreiner e20e679
wip: scaffold relative URL validation
Parkreiner df2e47e
refactor: remove unnecessary intermediary data types
Parkreiner 36ebd8d
fix: update script to be runnable from root directory
Parkreiner c3f998d
refactor: rename script
Parkreiner 9e48eb8
refactor: reorganize scripts again
Parkreiner 3b9ec5e
chore: finish initial version of validation script
Parkreiner 88f7be2
chore: set up initial version of CI
Parkreiner e035f1f
chore: beef up CI
Parkreiner 3b9c01e
fix: ensure relative avatars keep small scope
Parkreiner bc4bbda
fix: remove unnecessary matrix
Parkreiner abf9815
fix: update static files
Parkreiner affc506
refactor: split validation function into smaller pieces
Parkreiner 65fb7bc
refactor: standardize how errors are defined
Parkreiner 96fa5d4
refactor: apply majority of feedback
Parkreiner 5c45642
refactor: split off another function
Parkreiner ffd9861
refactor: extract pseudo-constants
Parkreiner bdf9c5f
refactor: update namespacing
Parkreiner 39b264a
refactor: split up package boundaries
Parkreiner a2c246e
refactor: split out error func
Parkreiner 50d651c
fix: update CI step
Parkreiner File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
name: CI | ||
on: | ||
pull_request: | ||
branches: [main] | ||
# Cancel in-progress runs for pull requests when developers push new changes | ||
concurrency: | ||
group: ${{ github.workflow }}-${{ github.ref }} | ||
cancel-in-progress: ${{ github.ref != 'refs/heads/main' }} | ||
jobs: | ||
validate-contributors: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: Check out code | ||
uses: actions/checkout@v4 | ||
- name: Set up Go | ||
uses: actions/setup-go@v5 | ||
with: | ||
go-version: "1.23.2" | ||
- name: Validate contributors | ||
run: go build ./scripts/contributors && ./contributors | ||
- name: Remove build file artifact | ||
run: rm ./contributors |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -134,3 +134,6 @@ dist | |
.yarn/build-state.yml | ||
.yarn/install-state.gz | ||
.pnp.* | ||
|
||
# Script output | ||
/contributors |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
module coder.com/coder-registry | ||
|
||
go 1.23.2 | ||
|
||
require gopkg.in/yaml.v3 v3.0.1 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,4 @@ | ||
gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405 h1:yhCVgyC4o1eVCa2tZl7eS0r+SDo693bJlVdllGtEeKM= | ||
gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0= | ||
gopkg.in/yaml.v3 v3.0.1 h1:fxVm/GzAzEWqLHuvctI91KS9hhNmmWOoWu0XTYJS7CA= | ||
gopkg.in/yaml.v3 v3.0.1/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM= |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,13 @@ | ||
--- | ||
display_name: Coder | ||
bio: Coder provisions cloud development environments via Terraform, supporting Linux, macOS, Windows, X86, ARM, Kubernetes and more. | ||
github: coder | ||
linkedin: https://www.linkedin.com/company/coderhq | ||
website: https://www.coder.com | ||
support_email: support@coder.com | ||
status: official | ||
--- | ||
|
||
# Coder | ||
|
||
Coder provisions cloud development environments via Terraform, supporting Linux, macOS, Windows, X86, ARM, Kubernetes and more. | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
--- | ||
display_name: Nataindata | ||
bio: Data engineer | ||
github: nataindata | ||
website: https://www.nataindata.com | ||
status: community | ||
--- |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This hasn't come up in any of the conversations yet, but I think that it's safe to give developers the freedom to throw whatever they want in the body. We'll just make sure the code always ignores it