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

Stub #4

Merged
merged 13 commits into from
Feb 15, 2018
Merged

Stub #4

merged 13 commits into from
Feb 15, 2018

Conversation

bam241
Copy link
Member

@bam241 bam241 commented Feb 13, 2018

Everything has been changed from cycamore::xxx to recycle::xxx.

The Cycamore archetypes are not deleted (yet) to make sure tests work properly.

Thanks to @bam241 for great help.

To address issue #1 , an additional PR should be placed deleting all the cycamore archetypes,

which shouldn't be too hard.

@bam241
Copy link
Member Author

bam241 commented Feb 13, 2018

@jbae11 I don't know what I did, but I got it to work :)

Actually I believe that circle-CI/recycle project need to be alive before a PR is issued to have the automatic hook working.

@bam241
Copy link
Member Author

bam241 commented Feb 13, 2018

ok actually I don't understand why this one don't trigger the test but #5 does :(

@bam241
Copy link
Member Author

bam241 commented Feb 13, 2018

might be because you where not in the cyclus group

@bam241 bam241 closed this Feb 13, 2018
@bam241 bam241 reopened this Feb 13, 2018
@jbae11
Copy link
Contributor

jbae11 commented Feb 15, 2018

@bam241 interesting.. This PR is made by you too, right? what do you suggest we do from here?

@bam241
Copy link
Member Author

bam241 commented Feb 15, 2018

first accept the invitation to join CYclus group member.
then close and reopen the PR ?

@bam241 bam241 merged commit 2f9e4a0 into cyclus:master Feb 15, 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