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

GitHub workflows #52

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

RohitRathore1
Copy link
Contributor

added github workflows for tests

@ChrisRackauckas
Copy link
Member

Those aren't needed on this repo.

@RohitRathore1
Copy link
Contributor Author

I thought that it will be helpful at the time of testing.

@ChrisRackauckas
Copy link
Member

What are the tests?

@RohitRathore1
Copy link
Contributor Author

What are the tests?

Yeah, you are right. In this repo we don't need this because there are no tests for performing.

@RohitRathore1
Copy link
Contributor Author

RohitRathore1 commented Jun 2, 2020

But in this case, tagbot will help us to create tags, releases, and changelogs for our Julia packages when it will register. And CompatHelper will help us to keep our [compat] entries up-to-date. Whenever one of our package's dependencies will release a new version, CompatHelper will open a pull request on our repository that will modifie our [compat] entry to reflect the newly released version.

@RohitRathore1
Copy link
Contributor Author

As the julia package DiffEqTutorials is registered because it has a good resource of tutorials and other things. We can also do the same for this repo. And now it's up to you. 🥰

@ChrisRackauckas
Copy link
Member

This repo would need a pretty substantial change for that to be useful IMO. I think there's a lot of issues with the reliance on notebooks that this approach has.

@RohitRathore1
Copy link
Contributor Author

Oh, I see but we can do those substantial changes. As mostly I would like to spend my time on Julia related projects. So, it will help me to deep dive also. 🤔

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