Skip to content

Fix capitalization of namespace #82

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 2 commits into from
Feb 15, 2023
Merged

Fix capitalization of namespace #82

merged 2 commits into from
Feb 15, 2023

Conversation

widhalmt
Copy link
Member

With some tools you get into problems when using differen capitalization for the namespace of the collection. While we establish to have it read "NETWAYS" wherever possible, I think a common, lowercase style for technical information might be beneficial.

@widhalmt widhalmt added bug Something isn't working documentation Improvements or additions to documentation labels Feb 14, 2023
@widhalmt widhalmt added this to the 1.0.0 milestone Feb 14, 2023
@widhalmt widhalmt self-assigned this Feb 14, 2023
mkayontour
mkayontour previously approved these changes Feb 14, 2023
Copy link
Member

@mkayontour mkayontour left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

everything looks fine to me.

@widhalmt
Copy link
Member Author

Looks like #55 needs to be merged before all tests can run through. Ngah.

@widhalmt widhalmt added this pull request to the merge queue Feb 15, 2023
Merged via the queue into main with commit e851626 Feb 15, 2023
@widhalmt widhalmt deleted the fix/capitalization branch February 15, 2023 14:40
ivareri pushed a commit to ivareri/ansible-collection-elasticstack that referenced this pull request Jun 17, 2025
* Allow installation of specific versions

fixes NETWAYS#81
ivareri pushed a commit to ivareri/ansible-collection-elasticstack that referenced this pull request Jun 17, 2025
ivareri pushed a commit to ivareri/ansible-collection-elasticstack that referenced this pull request Jun 17, 2025
* Fix capitalization of namespace

* Remove check for installed version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants