Skip to content

Commit

Permalink
docs: deploy docs when merging to master
Browse files Browse the repository at this point in the history
I believe the intention is to have the docs
aligned with what is on `master`, currently pushing on a release
branch will trigger a potential regression.

It could be reasonable for the docs to be aligned with the latest
release, but if that was the intention we should deploy the
docs as part of the release process, and avoid previous release branch
from deploying the docs.

I think this happened by mistake on
https://github.com/apache/superset/pull/27390/files#diff-1f3b6eaac4370772e5f0d6cebab7906e0a83ce78e07f6ae8a239c06b33a420d4R9
when the intention was to trigger CI on release branches, not to deploy
the docs.
  • Loading branch information
mistercrunch committed Apr 9, 2024
1 parent 93b83fe commit ecd3815
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion .github/workflows/superset-docs-deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ on:
- "docs/**"
branches:
- "master"
- "[0-9].[0-9]"

jobs:
config:
Expand Down

0 comments on commit ecd3815

Please sign in to comment.