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.

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

Remove all the MOODLE_INTERNAL fixture references #135

Open
stronk7 opened this issue Mar 23, 2024 · 0 comments
Open

Remove all the MOODLE_INTERNAL fixture references #135

stronk7 opened this issue Mar 23, 2024 · 0 comments

Comments

@stronk7
Copy link
Member

stronk7 commented Mar 23, 2024

Coming from #132, as far as moodle-cs is now an standalone product, we don't need these lines, previously required, in all the fixture files:

defined('MOODLE_INTERNAL') || die(); // Make this always the 1st line in all CS fixtures.

(or similar)

This is about to proceed to remove all them, not needed any more.

Note that there may be some MOODLE_INTERNAL uses in fixtures, sniffs and tests that can be legit (for the moodle.Files.MoodleInernal sniff). These need to remain unmodified.

Ciao :-)

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

No branches or pull requests

1 participant