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

fix: bad log #259

Merged
merged 1 commit into from
Apr 20, 2024
Merged

fix: bad log #259

merged 1 commit into from
Apr 20, 2024

Conversation

ComradeVanti
Copy link
Collaborator

For the remove and view command users should not specify package versions. I.e. they should not use these commands like openupm remove com.some.package@1.0.0. If they do specify a version, then a message warning the user should be printed.

This message is currently incorrect. If a user would use the command as above it would print please do not specify a version (Write only 'com.some.package@1.0.0').. This log also includes a version, despite that that is the exact issue.

Fixed. Now the log will say please do not specify a version (Write only 'com.some.package').

For the remove and view command users should not specify package versions. I.e. they should not use these commands like `openupm remove com.some.package@1.0.0`. If they do specify a version, then a message warning the user should be printed.

This message is currently incorrect. If a user would use the command as above it would print `please do not specify a version (Write only 'com.some.package@1.0.0').`. This log also includes a version, despite that that is the exact issue.

Fixed. Now the log will say `please do not specify a version (Write only 'com.some.package').`
@ComradeVanti ComradeVanti merged commit ac23e7b into master Apr 20, 2024
2 checks passed
@ComradeVanti ComradeVanti deleted the fix-bad-log branch April 20, 2024 12:08
github-actions bot pushed a commit that referenced this pull request Apr 20, 2024
## [1.22.2](1.22.1...1.22.2) (2024-04-20)

### Bug Fixes

* bad log ([#259](#259)) ([ac23e7b](ac23e7b))
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.

None yet

1 participant