[ 
https://issues.apache.org/jira/browse/YETUS-1127?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Allen Wittenauer updated YETUS-1127:
------------------------------------
    Release Note: 
<!-- markdown -->

Users:

Exclusion support has been modified to be slightly incompatible with previous 
releases:

* By default, `.yetus/excludes.txt` was supposed to have been read. However in 
previous releases it was ignored unless manually specified. 
*  `--excludes` has been restricted such that only files within the source 
repository, especially when running in `--docker` mode
* During a PR, the working set of changed files will now be completely 
recalculated, not just have additional files excluded.
* The `asflicense` plug-in will now honor the global excludes file unless 
`--asflicense-rat-globalexcludes=false` is passed.
* As a reminder, the excludes file needs to be regular expressions, not file 
globs. Therefore `.*` is the 'match everything' string.
* Several bugs with exclusion in various plug-ins have been simplified/cleaned 
up and should result in fewer corner-case issues.

Developers:

`yetus_file_to_array` has been modified to additionally strip out lines that 
contain spaces in addition to empty lines and lines that begin with a comment.

  was:
<!-- markdown -->

Users:

Exclusion support has been modified to be slightly incompatible with previous 
releases:

* By default, `.yetus/excludes.txt` was supposed to have been read. However in 
previous releases it was ignored unless manually specified. 
*  `--excludes` has been restricted such that only files within the source 
repository, especially when running in `--docker` mode
* During a PR, the working set of changed files will now be completely 
recalculated, not just have additional files excluded.
* The `asflicense` plug-in will now honor the global excludes file unless 
`--asflicense-rat-globalexcludes=false` is passed.
* As a reminder, the excludes file needs to be regular expressions, not file 
globs. Therefore `.*` is the 'match everything' string.

Developers:

`yetus_file_to_array` has been modified to additionally strip out lines that 
contain spaces in addition to empty lines and lines that begin with a comment.


> exclusions rework
> -----------------
>
>                 Key: YETUS-1127
>                 URL: https://issues.apache.org/jira/browse/YETUS-1127
>             Project: Yetus
>          Issue Type: Bug
>          Components: Precommit
>    Affects Versions: 0.14.0
>            Reporter: Allen Wittenauer
>            Assignee: Allen Wittenauer
>            Priority: Major
>             Fix For: 0.14.0
>
>
> Working on detsecrets, it is very clear that exclusions needs to have some 
> things re-thought.  In particular:
> a) comments in the file
> b) exclusions file itself should probably be an exclusions ....
> c) common code for routines to ignore files



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to