Skip to content

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 <kleinmrk@gmail.com>
@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.

3 participants