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

More work on the PGP Schemes #24

Merged
merged 3 commits into from
Jan 29, 2023
Merged

More work on the PGP Schemes #24

merged 3 commits into from
Jan 29, 2023

Conversation

wneessen
Copy link
Owner

  • Lots of code rework and cleanup
  • More test coverage
  • PGP/Inline is basically complete except for some tests

- Cleaned up and refactored a lot of code.
- Put the helper functions in its own file
- Added re-armoring, so we can add our comments/versions
- Fixed log message handling
- Put config specific stuff into its own file
- Updated go-mail, so we have access to embeds
- Added embedded file encryption
- Updated SPDX headers
- Cleaned up unneeded code in `pgpInline()` since we can only support plain text type
- Added more test coverage
@codecov-commenter
Copy link

Codecov Report

Merging #24 (469c348) into main (ae9c6de) will increase coverage by 4.13%.
The diff coverage is 82.18%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main      #24      +/-   ##
==========================================
+ Coverage   86.34%   90.47%   +4.13%     
==========================================
  Files           5        7       +2     
  Lines         410      441      +31     
==========================================
+ Hits          354      399      +45     
+ Misses         50       35      -15     
- Partials        6        7       +1     
Impacted Files Coverage Δ
openpgp/helper.go 64.77% <64.77%> (ø)
log/log.go 100.00% <100.00%> (ø)
openpgp/config.go 100.00% <100.00%> (ø)
openpgp/openpgp.go 100.00% <100.00%> (+29.80%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@wneessen wneessen merged commit b02050b into main Jan 29, 2023
@wneessen wneessen deleted the pgp_schemes branch January 29, 2023 21:06
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