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

Add serialized warnings array to JSON output #3588

Merged
merged 90 commits into from Oct 15, 2022

Conversation

MuluhGodson
Copy link
Contributor

@MuluhGodson MuluhGodson commented Apr 5, 2022

Issue

fixes #3099

Testing

cmake -B build -DENABLE_TESTS=ON
cd build
make run-gurka_warnings

What issue is this PR targeting? If there is no issue that addresses the problem, please open a corresponding issue and link it here.

Tasklist

  • Add tests
  • Add #fixes with the issue number that this PR addresses
  • Update the docs with any new request parameters or changes to behavior described
  • Update the changelog

Requirements / Relations

Link any requirements here. Other pull requests this PR is based on?

src/tyr/serializers.cc Outdated Show resolved Hide resolved
src/tyr/serializers.cc Outdated Show resolved Hide resolved
src/tyr/serializers.cc Outdated Show resolved Hide resolved
src/worker.cc Outdated Show resolved Hide resolved
src/worker.cc Outdated Show resolved Hide resolved
@nilsnolde nilsnolde added the pre-GSoC mandatory PRs to show commitment to GSoC project(s) label Apr 7, 2022
kevinkreiser
kevinkreiser previously approved these changes Oct 12, 2022
Co-authored-by: Kevin Kreiser <kevinkreiser@gmail.com>
@nilsnolde nilsnolde dismissed stale reviews from kevinkreiser and themself via 9b9ad42 October 12, 2022 14:19
Co-authored-by: Kevin Kreiser <kevinkreiser@gmail.com>
nilsnolde
nilsnolde previously approved these changes Oct 12, 2022
Copy link
Member

@nilsnolde nilsnolde left a comment

Choose a reason for hiding this comment

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

only missing changelog conflict update after the other PRs are merged into master

nilsnolde
nilsnolde previously approved these changes Oct 14, 2022
Copy link
Member

@nilsnolde nilsnolde left a comment

Choose a reason for hiding this comment

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

Resolved conflicts with master, should be good to merge now

kevinkreiser
kevinkreiser previously approved these changes Oct 14, 2022
@nilsnolde
Copy link
Member

Think I’ll try to revert the changes to gurka here before merging

nilsnolde
nilsnolde previously approved these changes Oct 14, 2022
Copy link
Member

@nilsnolde nilsnolde left a comment

Choose a reason for hiding this comment

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

Jeez, last time.. reverted the changes to gurka.cc and did a literal instead in the request.

@nilsnolde nilsnolde merged commit ae09f91 into valhalla:master Oct 15, 2022
@nilsnolde nilsnolde mentioned this pull request Oct 23, 2022
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pre-GSoC mandatory PRs to show commitment to GSoC project(s)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Warnings section in response
5 participants