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

code refactoring #45

Closed
wants to merge 531 commits into from
Closed

code refactoring #45

wants to merge 531 commits into from

Conversation

stahanna
Copy link
Collaborator

No description provided.

@bkhelifi
Copy link
Member

Thank you @stahanna.
I think that there is a commit issue in this PR:

  • there are the tutorials', download` folders that are not belonging to the website project. Can you remove them?
  • All other files (e.g. the song or the images) seem to be modified, but I am rather sure that this is not the case. @registerrier, do you understand how we can fix that with a git command?

@registerrier
Copy link
Collaborator

Well there are 530 commits in this PR, that's the full history of commits on the repo :-) I don't know how this is possible. I think closing and reopening a clean PR is the only way.

@stahanna
Copy link
Collaborator Author

@bkhelifi I have deleted the downloads and templates

@registerrier
Copy link
Collaborator

I think I understand. The PR is on the master branch, where the changes should be on gh-pages.
@stahanna close this PR, restart from a clean branch from gh-pages and open a new PR on gh-pages.

@bkhelifi There is a tutorials folder on the webpage. That's how we distribute the notebooks with gammapy download.

@bkhelifi
Copy link
Member

@registerrier, Hanna seems to have reverted the bad commits. Do you think that one can merge it?
This is so sad that one can not see the output before the merge. The use of JS complexifies the preview, I guess.

@registerrier
Copy link
Collaborator

@registerrier, Hanna seems to have reverted the bad commits. Do you think that one can merge it? This is so sad that one can not see the output before the merge. The use of JS complexifies the preview, I guess.

The PR is not on the correct branch!
It cannot be merged or we loose the all the configuration for binder that is on master for no change in the webpage that is on the gh-pages branch.

@bkhelifi
Copy link
Member

Well seen. I close the PR, @stahanna... And then you can restart from the clean version of the branch gh-pages of your fork (which is the current active primary branch), and create a new branch from it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

9 participants