Skip to content

Provide traits #14

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

Merged
merged 135 commits into from
Dec 12, 2021
Merged

Provide traits #14

merged 135 commits into from
Dec 12, 2021

Conversation

prince-chrismc
Copy link
Owner

@prince-chrismc prince-chrismc commented Sep 27, 2021

Testing PR

  • Test traits workflow

  • test boost install action

  • Add "linting" step for defaults

  • comment linting patch has incorrect whitespace (cant not be copied and applied locally)

  • clang-tidy and render-tests conflict is some cases (need to run the in tandem)

  • made badges to advertise supported JSON libraries

  • picojson example + badging

  • Security concerns about anyone running comment automation

@github-actions

This comment has been minimized.

2 similar comments
@github-actions
Copy link

👋 cmake-format: linting has some concerns

Diff

@github-actions
Copy link

👋 cmake-format: linting has some concerns

Diff

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@prince-chrismc
Copy link
Owner Author

//apply-linting

1 similar comment
@prince-chrismc
Copy link
Owner Author

//apply-linting

* 🔧 Replace RSA related deprecated functions (Thalhammer#189)

* RSA_size() --> EVP_PKEY_size()

* getting rid of deprecated functions in rsa-pss signature creation

* getting rid of deprecated functions in rsa-pss signature verification

* generate_hash not used in pss

* Update lint.yml

* Update lint.yml

* Update lint.yml

Co-authored-by: Marek Klein <[email protected]>
@prince-chrismc
Copy link
Owner Author

Do all comments waste jobs?

@prince-chrismc
Copy link
Owner Author

This should not run!

@prince-chrismc
Copy link
Owner Author

@prince-chrismc
Copy link
Owner Author

merging to master to make sure everything works on push

@prince-chrismc prince-chrismc merged commit a198367 into master Dec 12, 2021
@prince-chrismc prince-chrismc deleted the provide-traits branch December 12, 2021 19:50
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