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

docs: flank-scripts ops package organization #1583

Merged
merged 1 commit into from
Feb 10, 2021

Conversation

piotradamczyk5
Copy link
Contributor

Fixes #1542

This is a proposal on how to organize ops package inside flank-scripts

@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2021

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2021

Timestamp: 2021-02-09 17:24:30
Buildscan url for ubuntu-workflow run 552066483
https://gradle.com/s/idex2gaf3b2p4

For better code organization `updatebinaries` has a separate package inside `dependencies`, as well as `jfrog` in
`release` package.

Ops package structure and file organization is presented on tree below:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Ops package structure and file organization is presented on tree below:
Ops package structure and file organization are presented on the tree below:

Copy link
Contributor

@Sloox Sloox left a comment

Choose a reason for hiding this comment

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

Nice package organization

@mergify mergify bot merged commit 38849bc into master Feb 10, 2021
@mergify mergify bot deleted the #1542_flank-scripts_ops_organization branch February 10, 2021 08:53
@github-actions github-actions bot locked and limited conversation to collaborators Feb 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Redesign package structure for flank scripts domain
3 participants