Skip to content

Issue #3432109: Add new route /my-settings to redirect users to their settings page #2227

Issue #3432109: Add new route /my-settings to redirect users to their settings page

Issue #3432109: Add new route /my-settings to redirect users to their settings page #2227

Workflow file for this run

# This file contains custom lints that are not common to PHP/Drupal projects but
# are specific to how we want to build products at Open Social. These only run
# on pull requests since they are input for reviewer conversations and not hard
# rules such as our quality checks.
name: Best practices
on:
pull_request: { }
# We use the default concurrency grouping of allowing a single workflow per branch/PR/tag to run at the same time.
# In case of PRs we only care about the results for the last workflow run, so we cancel workflows already in progress
# when new code is pushed, in all other cases (branches/tags) we want to have a history for commits so it's easier to
# find breakages when they occur (head_ref is non-empty only when the workflow is triggered from a PR).
concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: ${{ github.head_ref != '' }}
defaults:
run:
shell: bash
jobs:
# Contrary to the other jobs we only perform this check on pull requests and
# accept that if a PR is merged despite this check we can ignore the addition
# on the main branch.
config_overrides:
name: No config overrides added
runs-on: ubuntu-latest
continue-on-error: true
steps:
- uses: actions/checkout@v4
with:
fetch-depth: 0
# We're only interested in config overrides being added.
# grep exits with 0 if it has matches, which we consider to be a fail
# so we invert.
- run: "! git diff ${{ github.event.pull_request.base.sha }} ${{ github.sha }} -- '**/*.services.yml' | grep -e '^+' | grep config.factory.override"