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

install normall requirements in the unit test ? #39

Open
12rambau opened this issue Dec 8, 2021 · 5 comments
Open

install normall requirements in the unit test ? #39

12rambau opened this issue Dec 8, 2021 · 5 comments

Comments

@12rambau
Copy link
Contributor

12rambau commented Dec 8, 2021

Instead of copy/pasting requirements from requirements.txt to requirements_build.txt would it be easier to install both of them ?
https://github.com/dfguerrerom/sepal_mgci/blob/09902478a7a7d1e255ae3c877f626c72f649a4bc/.github/workflows/unit.yml#L28

The build is currently failing

@dfguerrerom
Copy link
Collaborator

it will anyway fail because I don't have the google service account to do the tests, so I will wait until we have that.

@12rambau
Copy link
Contributor Author

12rambau commented Dec 8, 2021

that's on me right 😞 ?

@12rambau
Copy link
Contributor Author

Ok so lucky me, I have documented the process on GIS.stackexchange when I did it for sepal-ui.
Question do you want us to use your own account or would you prefer to use the sepal.classroom one ?

@dfguerrerom
Copy link
Collaborator

If we are planning to create tests in the sepal contrib modules then we probably want to use a generic account, but if that's something specific, perhaps is better to use an individual account.
In the meanwhile, I will use my personal account, thanks for sharing the process ;)

@12rambau
Copy link
Contributor Author

12rambau commented Dec 21, 2021

I've already completed the process from my side on the public account I'll share with you the credentials (mail I think), I also think it's better to group them (I just need to figure out how we want to order things in the cloud bucket).

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

No branches or pull requests

2 participants