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

NOT TO MERGE Crypto agility ATL-6707 #835

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

FabioPinheiro
Copy link
Contributor

@FabioPinheiro FabioPinheiro commented Apr 17, 2024

NOT TO MERGE

Deprecate in favor of #834
(this OR is just to compare changes and can be deleted)

crypto-agility-ATL-6707...input-output-hk:ATL-6924-add-support-for-other-keys

Copy link

Unit Test Results

0 tests   - 328   0 ✔️  - 327   0s ⏱️ -35s
0 suites  -   48   0 💤  -     1 
0 files    -   48   0 ±    0 

Results for commit 48a7856. ± Comparison against base commit 15e6865.

@atala-dev
Copy link
Contributor

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors
✅ REPOSITORY dustilock yes no
✅ REPOSITORY git_diff yes no
✅ REPOSITORY syft yes no

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

@EzequielPostan
Copy link
Contributor

I now can note that your PR is against main while the other PR you see from me is to open-source-node
We should decide how to manage the node changes once we open source it

@FabioPinheiro
Copy link
Contributor Author

I now can note that your PR is against main while the other PR you see from me is to open-source-node We should decide how to manage the node changes once we open source it

That a good point
I'm trying to copy (by hand) some on the work in this Branch to the another branch that is targeting ATL-6924-add-support-for-other-keys
In the meantime can I merge #836 into the branch open-source-node

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