Skip to content

use ForemanApi from apypie#1789

Merged
evgeni merged 1 commit into
developfrom
the-refactoring
Nov 26, 2024
Merged

use ForemanApi from apypie#1789
evgeni merged 1 commit into
developfrom
the-refactoring

Conversation

@evgeni

@evgeni evgeni commented Oct 28, 2024

Copy link
Copy Markdown
Member

includes #1797

@evgeni
evgeni force-pushed the the-refactoring branch 9 times, most recently from 4e78098 to 9a2b637 Compare October 28, 2024 11:49
@evgeni
evgeni force-pushed the the-refactoring branch 2 times, most recently from 3dd17c4 to ef6595f Compare October 28, 2024 12:44

@ekohl ekohl left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Consider this some of the notes I made while reading this. I realize it's very much a work in progress and should result in a real PR to apypie itself before updating the vendored copy.

Comment thread plugins/module_utils/_apypie.py Outdated
Comment thread plugins/module_utils/foreman_helper.py
Comment thread plugins/module_utils/_apypie.py Outdated
Comment thread plugins/module_utils/foreman_helper.py Outdated
@evgeni
evgeni force-pushed the the-refactoring branch 4 times, most recently from 8121f36 to 9adb24b Compare October 30, 2024 09:27
@evgeni
evgeni force-pushed the the-refactoring branch 4 times, most recently from 6dee41e to 94c0813 Compare November 21, 2024 09:19
@evgeni evgeni changed the title move some ansible-free code to apypie use ForemanApi from apypie Nov 21, 2024
@evgeni
evgeni marked this pull request as ready for review November 22, 2024 08:25
@evgeni
evgeni merged commit 879ec33 into develop Nov 26, 2024
@evgeni
evgeni deleted the the-refactoring branch November 26, 2024 08:42
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.

2 participants