Skip to content
This repository has been archived by the owner on May 14, 2020. It is now read-only.

Refactoring ModSecurity CRS Branch v3.0/master #1632

Closed
wants to merge 1 commit into from
Closed

Refactoring ModSecurity CRS Branch v3.0/master #1632

wants to merge 1 commit into from

Conversation

soufianebenali
Copy link
Contributor

These changes implement part 1 of the agreed refactoring explained in #1600:

  • Create a new folder tests in the root folder
  • Move util/regression-tests/ -> tests/regression/, and util/integration/ -> tests/integration
  • Rename folder documentation/ to docs/
  • Create folder examples/ and move crs-setup.conf.example -> examples/crs-setup.conf

@soufianebenali soufianebenali changed the title Refactoring tests folders Refactoring ModSecurity CRS Branch v3.0/master Nov 19, 2019
@fgsch
Copy link
Contributor

fgsch commented Nov 19, 2019

This is EOL. Not sure if we should push any further changes to this branch.

@fgsch fgsch closed this Nov 19, 2019
@fgsch fgsch reopened this Nov 19, 2019
@bittner
Copy link
Contributor

bittner commented Nov 20, 2019

Thank you for the feedback! That's good to know.

Should we close this PR unmerged?

@fgsch
Copy link
Contributor

fgsch commented Nov 20, 2019

Yup, close it please.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants