Skip to content

[WIP] Initial import of extraction from SecurityBundle #1

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

Closed
wants to merge 11 commits into from
Closed

[WIP] Initial import of extraction from SecurityBundle #1

wants to merge 11 commits into from

Conversation

linaori
Copy link
Contributor

@linaori linaori commented Aug 24, 2015

This PR is part of the initial import of what I've extracted from the SecurityBundle. I've made a todo list here. Who ever feels like helping out, please fork my branch (feature/extract-from-core) and make Pull Requests so they show up in this merge.

TODO List

  • Add README.md
  • Add .travis.yml
  • Add XSD
  • Update changelog
  • Update docs
  • Configure Travis
  • Add the license file: moved from Resources/meta/LICENSE to root
  • Minimize required dependencies in composer.json
  • Add a BC layer for the ACL configuration under security

If I'm missing anything, please let me know.

@linaori linaori changed the title [WIPInitial import of extraction from SecurityBundle [WIP] Initial import of extraction from SecurityBundle Aug 24, 2015
@fabpot
Copy link
Member

fabpot commented Aug 24, 2015

I've added this repo on Travis.

@@ -0,0 +1,23 @@
Security Bundle - ACL (Access Control List)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To be consistent, it should be named README.md

@linaori
Copy link
Contributor Author

linaori commented Aug 25, 2015

@nicolas-grekas can you verify the .travis.yml and see if anything is missing?

@fabpot
Copy link
Member

fabpot commented Aug 25, 2015

@iltar The LICENSE file should be put in the root directory (that's the new best practice now).


php:
- 5.6
- hhvm
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you remove the php section and add those version directly into the matrix?

@linaori
Copy link
Contributor Author

linaori commented Aug 25, 2015

@fabpot not sure if the matrix is the way you imagined it now; build 4.1 No language set

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

Successfully merging this pull request may close these issues.

3 participants