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

[ADD][9.0] travis: Add travis for v9 #7

Merged
merged 2 commits into from
Oct 16, 2016

Conversation

lasley
Copy link

@lasley lasley commented Oct 11, 2016

This PR adds travis file for v9

@lasley lasley mentioned this pull request Oct 11, 2016
3 tasks
@pedrobaeza
Copy link
Member

Maybe you can add also Transifex support

@lasley
Copy link
Author

lasley commented Oct 11, 2016

@pedrobaeza I was in the process of making an issue for this - I can't find the credentials in OCA/maintainer-quality-tools#194

If this is something you can take care of (or better, guide me to/on), I can add immediately 😄

@pedrobaeza
Copy link
Member

As a new repository, the secure is not created there, so we need to manually get it with gem instructions. Can you try it?

@lasley
Copy link
Author

lasley commented Oct 11, 2016

The instructions say your-password. To confirm, I just make something random here?

@lasley
Copy link
Author

lasley commented Oct 11, 2016

Seems to have worked. I'll submit that commit to 8.0 as well

@pedrobaeza
Copy link
Member

Uhm, we need to provide the OCA password, so it's not possible you to do it. Let me do it for you.

@lasley
Copy link
Author

lasley commented Oct 11, 2016

Oh so it didn't work? False 🍏 😢

@pedrobaeza
Copy link
Member

Uhm, I'm not sure if it's a false positive... Try with an invalid secure to see

@lasley
Copy link
Author

lasley commented Oct 11, 2016

Seems it is. Wonder if this is caused by OCA/maintainer-quality-tools#340 - @moylop260 any thoughts?

@lasley
Copy link
Author

lasley commented Oct 14, 2016

@pedrobaeza - I'm going to revert Transifex changes on this. Did you want to submit a PR to this branch with them, submit a new PR to the repo, or just drop the secret here and have me handle it?

@moylop260
Copy link

👍
The transifex error is not related with OCA/maintainer-quality-tools#340
We need the new secure password from @pedrobaeza
IMHO we can work it in other PR

@pedrobaeza
Copy link
Member

This is the correct secure:

secure: "P8zdckjpfLcTbz/kq8FNOgVMHjFNESk0H5/lHQQinBg7nr3oiWIg1tStroOJQtIn26+qWT2GZUX8V90FNCH6aGvqH8oIGguhZGibFHINarho2LjQd5DhhyIgm0qF12H31hFLOQokmlyx9r3A8Sw6EWPtWQ/vFBsL97ErhMK7AyX9vhDZCvpLX/zQyGpSXGNbogpmlhrV4z1RBBE9AtvPienTqkdz86a5wFOSjXM8Is5QVWZfKyFIxytiIxDERt+AS71FNXdrh73qRNbMIkULO+ROcdgGcMXdigiSZmny3n/kkty3I7lZO6K6bXLPugfii4Y2EiyJm+Mod99p9AAPAdifKbXF+ZJDWqkVbeooCBt93V1juQwsn1RT1Yms3U6F2lrw9oMm1M27ht+DSfcKMhBdtvDMN4tzU1zwxUx9kZ+oPmSDqSvPECIlUpHOvn4IUBEEuO1fHzooSX9scMFKJAx4Q7wsuPyUySFUy0J22AlbJcKcZOqtVw3++9WH1foxSWxMHjL/UjLSAn1B03xJAj3hALHMj61+R1HA/9v1/UCigwxkpbsMrWJK6SkXqBUcd5ECiHEScwNXr0nKLEydZOqorofPk4BjroGGyTd2h1iqa1suZ2zb3QJXcPvlsxzO/hvLm+wdzMyeJTe4I78kOkfGlmzSWJgoEq2ngad1rng="

@lasley
Copy link
Author

lasley commented Oct 16, 2016

Thanks @pedrobaeza - secure updated

@pedrobaeza
Copy link
Member

Merging then!

@pedrobaeza pedrobaeza merged commit 7be95f9 into OCA:9.0 Oct 16, 2016
@lasley lasley deleted the feature/9.0/travis branch October 16, 2016 17:12
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.

3 participants