Skip to content

replace RUN_SCRIPTS with ADMINISTER#256

Merged
jglick merged 2 commits intomasterfrom
replace-RUN_SCRIPTS-with-ADMINISTER
Jun 25, 2025
Merged

replace RUN_SCRIPTS with ADMINISTER#256
jglick merged 2 commits intomasterfrom
replace-RUN_SCRIPTS-with-ADMINISTER

Conversation

@jtnord
Copy link
Member

@jtnord jtnord commented Jun 24, 2025

replace RUN_SCRIPTS with ADMINISTER

RUN SCripts is deprecated and realms don;t grant it on its own without ADMINISTER as it really is ADMINISTER.

Just check ADMINISTER

Testing done

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests that demonstrate the feature works or the issue is fixed

replace RUN_SCRIPTS with ADMINISTER

RUN SCripts is deprecated and realms don;t grant it on its own without ADMINISTER as it really is ADMINISTER.

Just check ADMINISTER
@jtnord jtnord requested a review from a team as a code owner June 24, 2025 16:36
@jtnord jtnord requested review from jmdesprez and removed request for a team June 24, 2025 16:36
@jglick jglick added the bug label Jun 24, 2025
@jglick
Copy link
Member

jglick commented Jun 24, 2025

@jtnord jtnord requested a review from a team June 25, 2025 08:09
@jglick jglick merged commit 2191c4c into master Jun 25, 2025
17 checks passed
@jglick jglick deleted the replace-RUN_SCRIPTS-with-ADMINISTER branch June 25, 2025 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants