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

pre-publish methods should be private? #80

Open
7yl4r opened this issue Aug 2, 2018 · 5 comments
Open

pre-publish methods should be private? #80

7yl4r opened this issue Aug 2, 2018 · 5 comments
Labels

Comments

@7yl4r
Copy link
Member

7yl4r commented Aug 2, 2018

Good point raised by @mjm8 . Maybe this repo should be private?

I like leaving it public for collaborative, easy-of-use, and (potential future) cost reasons. However we can easily switch it to private once we have something in here we don't want public.

"DAG" definitions in general are going to be hard for someone outside of IMaRS to use without our help because they only describe the jobs at a high level. We can always develop individual software packages separately and then use them from within airflow (like the wv2-processing scripts).

@7yl4r 7yl4r added the question label Aug 2, 2018
@mjm8
Copy link
Collaborator

mjm8 commented Aug 2, 2018 via email

@7yl4r
Copy link
Member Author

7yl4r commented Aug 2, 2018

Ok. I think I may have misinterpreted your concerns.

You are saying that we don't want to publish something before it is done so that we can avoid sharing confusing or incomplete work?

This is handled (sort of) by "branches" in git.
The "test" branch (aka "development" branch) is for works-in-progress, but the "production" (aka "stable") branch is the public-facing version.

Both branches are publicly visible, but if we are sharing publicly we link to the "production" branch.
This is a pattern common to most software development efforts.

For this repo the branches are:

  • test/development is called "master"
  • production/stable is called "prod"

The branch names could easily be changed.

A good visual of how the branches are related is the "network graph".
Here is a more complex example of branches: nasa/CMR net graph. There they have dev branches for fixing particular issues which get merged into master (their production branch) after the issue is fixed. Then periodic numbered release branches pull from the master.

Does this address your concern or am I still missing something?

@mjm8
Copy link
Collaborator

mjm8 commented Aug 2, 2018 via email

@7yl4r
Copy link
Member Author

7yl4r commented Aug 2, 2018

I see. You are correct then.

To maximize protection from getting scooped we should make this repository private and release publicly elsewhere.

I do maintain that this repository will not contain anything worth scooping, but I will make it private to be on the safe side. I just need to check that @IMaRS-bot is set up on the nodes first.

@7yl4r 7yl4r self-assigned this Aug 2, 2018
@mjm8
Copy link
Collaborator

mjm8 commented Aug 2, 2018 via email

@7yl4r 7yl4r removed their assignment Nov 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants