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.

Sign up for GitHub

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

Jump to bottom

Implement note rule CPMR0061 - Id Contains "." #34

Closed
2 tasks done
Tracked by #30
AdmiringWorm opened this issue Jan 28, 2024 · 1 comment · Fixed by #49
Closed
2 tasks done
Tracked by #30

Implement note rule CPMR0061 - Id Contains "." #34

AdmiringWorm opened this issue Jan 28, 2024 · 1 comment · Fixed by #49
Assignees
@AdmiringWorm
Labels
5 - Released The issue has been resolved, and released to the public for consumption Improvement Issues that enhances existing functionality, or adds new features
Milestone
0.2.0

Comments

@AdmiringWorm
Copy link
Member

Checklist

  • I have verified this is the correct repository for opening this issue.
  • I have verified no other issues exist related to my request.

Is Your Feature Request Related To A Problem? Please describe.

No response

Describe The Solution. Why is it needed?

The package validator note rule should be implemented, this should flag any identifiers that uses a full stop, and it is not .install, .portable, .commandline, .extension, .template or .powershell.

This rule should be made more stricter than the implementation in package validator.

Additional Context

Rule Documentation: https://ch0.co/rules/cpmr0061

The documentation is outdated when it comes to the acceptable suffixes, and needs to be updated in the docs repository.

Related Issues

No response
@AdmiringWorm AdmiringWorm added Improvement Issues that enhances existing functionality, or adds new features 0 - Backlog Issue is accepted, but is not ready to be worked on or not in current sprint labels Jan 28, 2024
@AdmiringWorm AdmiringWorm added this to the 0.2.0 milestone Jan 28, 2024
@AdmiringWorm AdmiringWorm mentioned this issue Jan 28, 2024
Closed
8 tasks
@AdmiringWorm AdmiringWorm added 1 - Ready Issue is accepted, milestone added and are ready to be worked on and removed 0 - Backlog Issue is accepted, but is not ready to be worked on or not in current sprint labels Jan 28, 2024
@AdmiringWorm AdmiringWorm self-assigned this Jan 28, 2024
@AdmiringWorm AdmiringWorm added 2 - Working A user or team member has started working on the issue and removed 1 - Ready Issue is accepted, milestone added and are ready to be worked on labels Jan 29, 2024
@AdmiringWorm AdmiringWorm added 0 - Backlog Issue is accepted, but is not ready to be worked on or not in current sprint 1 - Ready Issue is accepted, milestone added and are ready to be worked on and removed 2 - Working A user or team member has started working on the issue 0 - Backlog Issue is accepted, but is not ready to be worked on or not in current sprint labels Apr 25, 2024
@AdmiringWorm AdmiringWorm added 2 - Working A user or team member has started working on the issue and removed 1 - Ready Issue is accepted, milestone added and are ready to be worked on labels May 21, 2024
AdmiringWorm added a commit to AdmiringWorm/chocolatey-community-validation that referenced this issue Jun 7, 2024
@AdmiringWorm
(chocolatey-community#34) Implement note rule CPMR0061 - Id Contains "."
d20f10e 
This implements the note rule CPMR0061 that verifies whether a package
uses a dot in its identifier, which is not recommended to do unless
absolutely necessary.

This implementation is more comprehensive than what is implemented in
package validator, where package validator will have false negatives
that this implementation will properly flag.
@AdmiringWorm AdmiringWorm mentioned this issue Jun 7, 2024
Merged
10 tasks
@AdmiringWorm AdmiringWorm added 3 - Review Code has been added, and is available for review as a pull request and removed 2 - Working A user or team member has started working on the issue labels Jun 7, 2024
AdmiringWorm added a commit to AdmiringWorm/chocolatey-community-validation that referenced this issue Jul 5, 2024
@AdmiringWorm
(chocolatey-community#34) Implement note rule CPMR0061 - Id Contains "."
bb2942d 
This implements the note rule CPMR0061 that verifies whether a package
uses a dot in its identifier, which is not recommended to do unless
absolutely necessary.

This implementation is more comprehensive than what is implemented in
package validator, where package validator will have false negatives
that this implementation will properly flag.
AdmiringWorm added a commit to AdmiringWorm/chocolatey-community-validation that referenced this issue Jul 8, 2024
@AdmiringWorm
(chocolatey-community#34) Implement note rule CPMR0061 - Id Contains "."
819f526 
This implements the note rule CPMR0061 that verifies whether a package
uses a dot in its identifier, which is not recommended to do unless
absolutely necessary.

This implementation is more comprehensive than what is implemented in
package validator, where package validator will have false negatives
that this implementation will properly flag.
@gep13 gep13 closed this as completed in #49 Jul 8, 2024
gep13 added a commit that referenced this issue Jul 8, 2024
@gep13
Merge pull request #49 from AdmiringWorm/34-rule-cpmr0061
3450719 
(#34) Implement note rule CPMR0061 - Id Contains "."
@AdmiringWorm AdmiringWorm added 4 - Done Code has been added to the repository, and has been reviewed by a team member and removed 3 - Review Code has been added, and is available for review as a pull request labels Jul 8, 2024
AdmiringWorm added a commit to AdmiringWorm/chocolatey-community-validation that referenced this issue Dec 11, 2024
@AdmiringWorm
Merge branch 'release/0.2.0'
e88ef08 
* release/0.2.0: (21 commits)
  (maint) Fix incorrect label used in config
  (chocolatey-community#58) Implement requirement rule CPMR0074
  (chocolatey-community#28) Add rule to disallow icon URL using GitHub links
  (chocolatey-community#37) Implement note rule CPMR0068
  (chocolatey-community#36) Implement note rule CPMR0067
  (maint) Set explicit PlatformTarget for console
  (maint) synced local '.github/ISSUE_TEMPLATE/' with remote '.github/ISSUE_TEMPLATE/'
  (chocolatey-community#38) Add rule for validating ID length
  (chocolatey-community#39) Add rule for validating underscore in ID
  (chocolatey-community#34) Implement note rule CPMR0061 - Id Contains "."
  (maint) synced local '.github/PULL_REQUEST_TEMPLATE.md' with remote '.github/PULL_REQUEST_TEMPLATE.md'
  (build) Update Chocolatey.Cake.Recipe to latest
  (chocolatey-community#35) Implement note rule CPMR0062
  (build) Update to latest Recipe package
  (maint) synced local '.github/ISSUE_TEMPLATE/' with remote '.github/ISSUE_TEMPLATE/'
  (chocolatey-community#32) Update Help URLs to short link
  (chocolatey-community#29) Add missing release notes to generated packages
  (maint) Update extension name in Readme
  (maint) synced local '.templates/' with remote '.github/GitReleaseManager/.templates/'
  (chocolatey-community#26) Use milestone for license URL replacement
  ...
@AdmiringWorm AdmiringWorm added 5 - Released The issue has been resolved, and released to the public for consumption and removed 4 - Done Code has been added to the repository, and has been reviewed by a team member labels Dec 11, 2024
@AdmiringWorm
Copy link
Member Author

🎉 This issue has been resolved in version 0.2.0 🎉

The release is available on:

Your GitReleaseManager bot 📦 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AdmiringWorm AdmiringWorm

Labels
5 - Released The issue has been resolved, and released to the public for consumption Improvement Issues that enhances existing functionality, or adds new features
Projects
None yet
Milestone
0.2.0
Development

Successfully merging a pull request may close this issue.

(#34) Implement note rule CPMR0061 - Id Contains "." AdmiringWorm/chocolatey-community-validation
1 participant
@AdmiringWorm