style: Reorg repo structure and add CODEOWNERS #19
Conversation
This file contains 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
There was a problem hiding this comment.
Copilot reviewed 137 out of 138 changed files in this pull request and generated 1 comment.
Files not reviewed (1)
- .github/CODEOWNERS: Language not supported
Comments suppressed due to low confidence (2)
CHANGELOG.md:1
- The removal of CHANGELOG entries may lead to loss of project version history. If this deletion is unintentional, consider archiving the old entries instead of removing them completely.
## 1.0.0 (Month Date, Year) ... Initial release of this project.
.github/workflows/ossf_scorecard.yml:11
- The branch filter was reduced to only 'main', removing 'master'. Please verify that this change reflects the current branch usage for the repository.
branches: [main]
README.md
Outdated
| |NGINX SOAP REST|Example NGINX configuration to translate between SOAP and REST| | ||
| |Title|Description|Owner| | ||
| |-----|-----------|-----| | ||
| |NGINX Advanced Healthcheck|Advanced active healthchecks for NGINX Plus|| |
There was a problem hiding this comment.
The Owner column for the 'NGINX Advanced Healthcheck' demo is empty. Please add the appropriate CODEOWNER or a placeholder if no owner is designated.
Suggested change
| |NGINX Advanced Healthcheck|Advanced active healthchecks for NGINX Plus|| | |
| |NGINX Advanced Healthcheck|Advanced active healthchecks for NGINX Plus|TBD| |
Copilot is powered by AI, so mistakes are possible. Review output carefully before use.
alessfg
force-pushed
the
structure-codeowners
branch
2 times, most recently
from
708e4de to
ddb3db3
Compare
There was a problem hiding this comment.
Copilot reviewed 115 out of 116 changed files in this pull request and generated no comments.
Files not reviewed (1)
- .github/CODEOWNERS: Language not supported
Comments suppressed due to low confidence (1)
.github/workflows/ossf_scorecard.yml:11
- Limiting branch filtering to 'main' may exclude updates from the 'master' branch if it is still in use. Please verify that this change is intentional.
+ branches: [main]
alessfg
force-pushed
the
structure-codeowners
branch
from
ddb3db3 to
3ca900e
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Proposed changes
Reorganized the overall repo structure so that demos are categorized by product offering; and add explicit CODEOWNERS for each demo.
Checklist
Before creating a PR, run through this checklist and mark each as complete:
README.mdand/orCHANGELOG.md).