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

Block scoped usage #235

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Block scoped usage #235

wants to merge 3 commits into from

Conversation

uhafner
Copy link
Member

@uhafner uhafner commented Oct 22, 2019

@jenkinsci jenkinsci deleted a comment Oct 22, 2019
@uhafner uhafner added the enhancement Enhancement of existing functionality label Oct 25, 2019
@PhilippHomann
Copy link

@uhafner Any plans on moving this forward?

@uhafner
Copy link
Member Author

uhafner commented Oct 12, 2020

I stopped working on it since I found no solution to return the results back to the master. The basic use case is working, just the master slave communication is missing. Interested in helping?

@alexanderLinear
Copy link

any progress?

@uhafner
Copy link
Member Author

uhafner commented Jul 30, 2022

Actually, I improved the PR a little bit (https://github.com/jenkinsci/warnings-ng-plugin/tree/block-scoped). I got the serialization working, but then I noticed that in order to correctly support it in the planned way I need to change the API of the analysis-model from a text oriented input to a line oriented input, which is not trivial. At least a lot of existing parsers work differently (e.g. read multiple lines of the console or read XML files).

So I am sorry to say this PR still is stalled.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement of existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants