Add ability to ignore warnings

README.md

@@ -37,6 +37,7 @@ jobs:
  - run: ./gradlew lint
  - uses: yutailang0119/action-android-lint@v3
  with:
+ ignore-warnings: true # Ignore Lint Warnings
  report-path: build/reports/*.xml # Support glob patterns by https://www.npmjs.com/package/@actions/glob
  continue-on-error: false # If annotations contain error of severity, action-android-lint exit 1.
 ```

__tests__/main.test.ts

@@ -7,6 +7,7 @@ import {expect, test} from '@jest/globals'
 test('test runs', () => {
  process.env['INPUT_REPORT-PATH'] = path.join(__dirname, 'resource', '*.xml')
  process.env['INPUT_FOLLOW-SYMBOLIC-LINKS'] = 'true'
+ process.env['INPUT_IGNORE-WARNINGS'] = 'false'
  const np = process.execPath
  const ip = path.join(__dirname, '..', 'lib', 'main.js')
  const options: cp.ExecFileSyncOptions = {
@@ -30,6 +31,7 @@ test('test runs without error', () => {
  'empty-results.xml'
  )
  process.env['INPUT_FOLLOW-SYMBOLIC-LINKS'] = 'true'
+ process.env['INPUT_IGNORE-WARNINGS'] = 'false'
  const np = process.execPath
  const ip = path.join(__dirname, '..', 'lib', 'main.js')
  const options: cp.ExecFileSyncOptions = {

__tests__/parser.test.ts

@@ -22,7 +22,22 @@ test('test parseXmls', () => {
  44
  )
 
- expect(parseXmls([file1, file2])).resolves.toEqual([annotation1, annotation2])
+ expect(parseXmls([file1, file2], false)).resolves.toEqual([annotation1, annotation2])
+})
+
+test('test parseXmls and ignore warnings', () => {
+ const file1 = path.join(__dirname, 'resource', 'lint-results.xml')
+ const file2 = path.join(__dirname, 'resource', 'empty-results.xml')
+
+ const annotation2 = new Annotation(
+ 'Error',
+ 'Ignoring results: The result of `subscribe` is not used',
+ 'Foo.kt',
+ 33,
+ 44
+ )
+
+ expect(parseXmls([file1, file2], true)).resolves.toEqual([annotation2])
 })
 
 test('test parseXml with issues', () => {
@@ -52,13 +67,13 @@ test('test parseXml with issues', () => {
  44
  )
 
- expect(parseXml(xml)).resolves.toEqual([annotation])
+ expect(parseXml(xml, false)).resolves.toEqual([annotation])
 })
 
 test('test parseXml without issue', () => {
  const xml = `<?xml version="1.0" encoding="UTF-8"?>
  <issues format="6" by="lint 7.2.1">
  </issues>`
 
- expect(parseXml(xml)).resolves.toEqual([])
+ expect(parseXml(xml, false)).resolves.toEqual([])
 })

action.yml

@@ -8,6 +8,9 @@ inputs:
  follow-symbolic-links:
  description: 'Indicates whether to follow symbolic links'
  default: true
+ ignore-warnings:
+ description: 'Ignore Lint Warnings'
+ default: false 
 runs:
  using: 'node16'
  main: 'dist/index.js'

src/main.ts

@@ -6,13 +6,14 @@ import {parseXmls} from './parser'
 async function run(): Promise<void> {
  try {
  const reportPath = core.getInput('report-path', {required: true})
+ const ignoreWarnings = core.getBooleanInput('ignore-warnings')
  const globOptions = {
  followSymbolicLinks: core.getBooleanInput('follow-symbolic-links')
  }
  const globber = await glob.create(reportPath, globOptions)
  const files = await globber.glob()
 
- const annotations = await parseXmls(files)
+ const annotations = await parseXmls(files, ignoreWarnings)
 
  echoMessages(annotations)
 

src/parser.ts

@@ -3,17 +3,23 @@ import * as core from '@actions/core'
 import * as xml2js from 'xml2js'
 import {Annotation} from './annotation'
 
-export const parseXmls = async (files: string[]): Promise<Annotation[]> => {
+export const parseXmls = async (
+ files: string[],
+ ignoreWarnings: boolean
+): Promise<Annotation[]> => {
  const list = await Promise.all(
  files.map(async file => {
  const xml = fs.readFileSync(file, 'utf-8')
- return await parseXml(xml)
+ return await parseXml(xml, ignoreWarnings)
  })
  )
  return list.flat()
 }
 
-export const parseXml = async (text: string): Promise<Annotation[]> => {
+export const parseXml = async (
+ text: string,
+ ignoreWarnings: boolean
+): Promise<Annotation[]> => {
  const parser = new xml2js.Parser()
  const xml = await parser.parseStringPromise(text)
  if (xml.issues.issue === undefined) return []
@@ -22,6 +28,9 @@ export const parseXml = async (text: string): Promise<Annotation[]> => {
  const annotations: Annotation[] = []
  for (const issueElement of xml.issues.issue) {
  const issue = issueElement.$
+ if (ignoreWarnings === true && issue.severity !== 'Error') {
+ continue;
+ }
  for (const locationElement of issueElement.location) {
  const location = locationElement.$