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

Remove deprecated servicenow.servicenow from Ansible 9 #254

Merged
merged 4 commits into from
Jun 24, 2023

Conversation

mariolenz
Copy link
Collaborator

See ansible-community/community-topics#246 for more information.

@@ -99,7 +99,6 @@ purestorage.flasharray: >=1.18.0,<2.0.0
purestorage.flashblade: >=1.11.0,<2.0.0
purestorage.fusion: >=1.4.0,<2.0.0
sensu.sensu_go: >=1.13.0,<2.0.0
servicenow.servicenow: >=1.0.0,<2.0.0
Copy link
Contributor

Choose a reason for hiding this comment

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

The collection MUST NOT be removed from Ansible 8. This is a breaking change and thus not acceptable since Ansible sticks to semantic versioning.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I don't think this is the only place we have to change in this case... let me have a look and do another commit.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@felixfontein I think it should be OK now.

@mariolenz mariolenz force-pushed the remove_servicenow branch from a844959 to dbaaa9e Compare June 23, 2023 16:05
@mariolenz mariolenz changed the title [WIP] Remove deprecated servicenow.servicenow from 8.2 [WIP] Remove deprecated servicenow.servicenow from Ansible 9 Jun 23, 2023
@mariolenz mariolenz requested a review from felixfontein June 23, 2023 16:10
@mariolenz
Copy link
Collaborator Author

@felixfontein BTW if you think this change is OK now and there's no need to discuss it in the next meeting, feel free to remove the [WIP] and merge it ;-)

Copy link
Contributor

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

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

Two small changes, after that we can merge it.

8/changelog.yaml Outdated Show resolved Hide resolved
9/changelog.yaml Outdated Show resolved Hide resolved
Co-authored-by: Felix Fontein <felix@fontein.de>
@mariolenz mariolenz changed the title [WIP] Remove deprecated servicenow.servicenow from Ansible 9 Remove deprecated servicenow.servicenow from Ansible 9 Jun 24, 2023
@felixfontein felixfontein merged commit f178686 into ansible-community:main Jun 24, 2023
@felixfontein
Copy link
Contributor

Thanks a lot for fixing this! I really wonder how that collection managed to sneak back in...

@mariolenz mariolenz deleted the remove_servicenow branch June 24, 2023 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants