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

3.x/features #1

Closed
wants to merge 8 commits into from
Closed

Conversation

earcanal
Copy link

This one probably does need either code review prior to merge, and/or an way of maintaining multiple versions.

@vsoch
Copy link
Contributor

vsoch commented Aug 17, 2018

Don't you think this needs versioning? This is a more used task, and I would like to have versioning here, especially given your reference of "3.x"

@earcanal
Copy link
Author

Yes. How do we manage versions?

@vsoch
Copy link
Contributor

vsoch commented Aug 24, 2018

Ah let's get this guy hooked up to Circle CI! For versions I say we should have the merge (that deploys the container with the experiment) have a version corresponding to the commit. Let me turn on CircleCI and see what happens...

config.json Outdated Show resolved Hide resolved
@vsoch
Copy link
Contributor

vsoch commented Sep 4, 2018

@earcanal I'm going to do the same as I did for the breath counting and start with this branch and make a crapton of changes for the CI - sound good?

@vsoch
Copy link
Contributor

vsoch commented Sep 4, 2018

okay let's pick this up in #2

Copy link
Contributor

@vsoch vsoch left a comment

Choose a reason for hiding this comment

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

Hey please PR to the branch I started for us to work on, ty!

@earcanal earcanal mentioned this pull request Sep 5, 2018
@vsoch
Copy link
Contributor

vsoch commented Sep 5, 2018

Please move work to expfactory-experiments/attention-network-test#1, this is the wrong task to PR to, closing.

@vsoch vsoch closed this Sep 5, 2018
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