Add missing files to release artifacts by falbrechtskirchinger · Pull Request #3728 · nlohmann/json

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Comments

Conversation

@falbrechtskirchinger

Copy link

Contributor

@falbrechtskirchinger falbrechtskirchinger commented

Sep 7, 2022

edited

Loading

Add missing files to include.zip:

  • amalgamated json_fwd.hpp
  • Bazel build files

Fixes #3727.

@falbrechtskirchinger

Copy link

Contributor Author

falbrechtskirchinger commented

Sep 7, 2022

edited

Loading

$ make include.zip
(...)
  adding: single_include/nlohmann/json_fwd.hpp (deflated 74%)
  adding: BUILD.bazel (deflated 81%)
  adding: WORKSPACE.bazel (stored 0%)

@falbrechtskirchinger falbrechtskirchinger marked this pull request as draft

September 7, 2022 07:22

@falbrechtskirchinger

Copy link

Contributor Author

(Let's wait on #3709.)

@coveralls

Copy link

coveralls commented

Sep 7, 2022

edited

Loading

Coverage Status

Coverage remained the same at 100.0% when pulling 2b3a1d4 on falbrechtskirchinger:add-missing-release-files into f4658de on nlohmann:develop.

@falbrechtskirchinger falbrechtskirchinger marked this pull request as ready for review

September 20, 2022 06:52

@nlohmann nlohmann added this to the Release 3.11.3 milestone

Sep 25, 2022
Copy link

Owner

@nlohmann nlohmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@nlohmann nlohmann merged commit 17d9eac into nlohmann:develop

Sep 25, 2022

@nlohmann

Copy link

Owner

Thanks!

@falbrechtskirchinger falbrechtskirchinger deleted the add-missing-release-files branch

September 25, 2022 11:55

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nlohmann nlohmann nlohmann approved these changes

+1 more reviewer

@gregmarr gregmarr gregmarr approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Projects

None yet

Milestone

Release 3.11.3

Development

Successfully merging this pull request may close these issues.

Amalgated json_fwd.hpp not included in include.zip

4 participants

@falbrechtskirchinger @coveralls @nlohmann @gregmarr