Skip to content

Add a cleanup task to the Ansible role#169

Merged
trz42 merged 7 commits intoEESSI:mainfrom
bedroge:add_cleanup_step
Apr 3, 2023
Merged

Add a cleanup task to the Ansible role#169
trz42 merged 7 commits intoEESSI:mainfrom
bedroge:add_cleanup_step

Conversation

@bedroge
Copy link
Copy Markdown
Collaborator

@bedroge bedroge commented Mar 7, 2023

This can be used to remove stuff at the end of the installation, currently it will just remove a list of specified packages (Go, only used for building direnv).

trz42
trz42 previously approved these changes Mar 30, 2023
Copy link
Copy Markdown
Contributor

@trz42 trz42 left a comment

Choose a reason for hiding this comment

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

Looks good.

Not yet tested.

trz42
trz42 previously approved these changes Apr 3, 2023
Copy link
Copy Markdown
Contributor

@trz42 trz42 left a comment

Choose a reason for hiding this comment

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

Resolved conflicts. Looks good. Can be merged after all GHA tests have passed.

missing new line at end of file
Remove trailing whitespaces.
Copy link
Copy Markdown
Contributor

@trz42 trz42 left a comment

Choose a reason for hiding this comment

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

All tests pass. 🎉

@trz42 trz42 merged commit 45db4ab into EESSI:main Apr 3, 2023
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