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

Fix/brokentests #39

Merged
merged 2 commits into from
Jan 8, 2022
Merged

Conversation

chadgates
Copy link
Contributor

#37

Updated certificates (10 years validity) for tests.

Fixed mendelson tests by using public key from mendelson test environment for partner mecas2 to fix #37 and recreated files:

mendelsonAS2_mecas2
mendelsonAS2_pyas2lib

@abhishek-ram
Copy link
Owner

I guess travis ci is no longer working, should we setup github actions?

@chadgates
Copy link
Contributor Author

I had migrated to travis-ci.com for a quick fix, but github actions would surely be ok as well.

I followed this: https://oliverde8.medium.com/how-to-migrate-from-travis-ci-org-to-com-5fbcfa541777

@chadgates
Copy link
Contributor Author

I am preparing the branches for the next PRs, but building on top of each other, as decoupling them won't make sense as Black update also made changes to test files. Either way, the build for this PR breaks without the pytest, pylama, black updates, due to package dependencies (typed-ast).

@abhishek-ram
Copy link
Owner

I had migrated to travis-ci.com for a quick fix, but github actions would surely be ok as well.

I followed this: https://oliverde8.medium.com/how-to-migrate-from-travis-ci-org-to-com-5fbcfa541777

The new service seems to be paid, let me setup actions over the weekend

@abhishek-ram
Copy link
Owner

I am preparing the branches for the next PRs, but building on top of each other, as decoupling them won't make sense as Black update also made changes to test files. Either way, the build for this PR breaks without the pytest, pylama, black updates, due to package dependencies (typed-ast).

Alright

@chadgates
Copy link
Contributor Author

@abhishek-ram
Copy link
Owner

Merging with master so that I can setup the tests

@abhishek-ram abhishek-ram merged commit 66954dc into abhishek-ram:master Jan 8, 2022
@abhishek-ram
Copy link
Owner

Actions have been added now...

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.

Tests do not run - expired certificates
2 participants