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.

Sign up for GitHub

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

Jump to bottom

add compiler problem matcher #286

Open
Trass3r opened this issue Jan 25, 2022 · 3 comments · May be fixed by #562
Open

add compiler problem matcher #286

Trass3r opened this issue Jan 25, 2022 · 3 comments · May be fixed by #562
Labels
feature request New feature or request to improve the current logic

Comments

@Trass3r
Copy link

Trass3r commented Jan 25, 2022

I'm puzzled this doesn't set up problem matchers for compiler messages like

/home/runner/work/user/repo/src/[...].java:203: warning: [dep-ann] deprecated item is not annotated with @Deprecated
    protected String getResource() {
/home/runner/work/user/repo/src/[...].java:563: error: cannot find symbol

Something like

		{
			"owner": "java",
			"pattern": [
				{
					"regexp": "^([^:]+):(\\d+): (warning|error): (.+?)$",
					"file": 1,
					"line": 2,
					"severity": 3,
					"message": 4
				}
			]
		}
@Trass3r Trass3r added feature request New feature or request to improve the current logic needs triage labels Jan 25, 2022
@dmitry-shibanov
Copy link
Contributor

Hello @Trass3r. Thank you for your report. We'll investigate the feature request. Could you please provide a public repository for testing ?

@Trass3r
Copy link
Author

Trass3r commented Jan 26, 2022 •
edited

Seen at https://github.com/tonihele/OpenKeeper/runs/4941368760.

@Trass3r
Copy link
Author

Trass3r commented Apr 13, 2023

See https://github.com/Trass3r/setup-cpp/blob/master/main.js for reference.

Trass3r added a commit to Trass3r/OpenKeeper that referenced this issue Nov 30, 2023
@Trass3r
report build warnings in CI
8145ea2 
see actions/setup-java#286
@Trass3r Trass3r linked a pull request Dec 1, 2023 that will close this issue
Open
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature request New feature or request to improve the current logic
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

add javac problem matcher Trass3r/setup-java
3 participants
@Trass3r @dmitry-shibanov @IvanZosimov