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

Feature ansible galaxy #30

Merged
merged 6 commits into from
Jun 17, 2024
Merged

Feature ansible galaxy #30

merged 6 commits into from
Jun 17, 2024

Conversation

SquirrelDeveloper
Copy link
Collaborator

No description provided.

manu added 6 commits June 10, 2024 09:37
This update includes the creation of a new Ansible Galaxy collection feature, which is responsible for parsing and handling data from the Ansible Galaxy API. It also introduces a form for playbook creation in the client's Playbooks page. The 'ExtraVarsViewEditor' component was refactored and moved into a new components directory. Various services and routes were updated to handle these changes properly.
In this commit, we made alterations to the import paths in multiple files to reflect changes in the structure of the shared library. This includes changes in enums and types that were previously segregated, but are now under a single namespace. Therefore, instead of multiple import statements, we now use a single import reference. This makes the code cleaner and easier to maintain.
async function installAnsibleGalaxyCollection(name: string, namespace: string) {
try {
logger.info('[SHELL] - installAnsibleGalaxyCollection Starting...');
const result = shell.exec(AnsibleGalaxyCmd.getInstallCollectionCmd(name, namespace));

Check failure

Code scanning / CodeQL

Uncontrolled command line Critical

This command line depends on a
user-provided value
.
@SquirrelDeveloper SquirrelDeveloper merged commit 137817a into master Jun 17, 2024
7 of 8 checks passed
@SquirrelDeveloper SquirrelDeveloper deleted the feature-ansible-galaxy branch June 17, 2024 18:19
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.

1 participant