potiuk commented on PR #761:
URL: https://github.com/apache/airflow-site/pull/761#issuecomment-1512107133
I **hope** it's going to work.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the
potiuk commented on PR #761:
URL: https://github.com/apache/airflow-site/pull/761#issuecomment-1512106636
Uploaded : https://pypi.org/project/sphinx-airflow-theme/0.0.12/ , Merging.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
potiuk commented on PR #761:
URL: https://github.com/apache/airflow-site/pull/761#issuecomment-1512065121
Coool! One more thing - can you please update version of the "sphinx-theme"
package @vincbeck - then I **think** I should be able to publish it to PyPI (I
think I have permissions :D).
potiuk commented on PR #761:
URL: https://github.com/apache/airflow-site/pull/761#issuecomment-1511624718
> @potiuk I am not able to locate this `.whl` file. Any hint on where I can
find it?
You should build it - I think one of the steps (with site.sh script does it)
potiuk commented on PR #761:
URL: https://github.com/apache/airflow-site/pull/761#issuecomment-1508627822
> So I think the issue you had was basically that the original script is
moved :)
I hope the result of this exercise will also be updating the docs :D
--
This is an automated
potiuk commented on PR #761:
URL: https://github.com/apache/airflow-site/pull/761#issuecomment-1508624074
BTW. @vincbeck I think it should be **just** enough to copy the theme to
/files folder, enter install the theme from there `pip install
/files/your.whl` and then run the "
potiuk commented on PR #761:
URL: https://github.com/apache/airflow-site/pull/761#issuecomment-1507447346
BTW., The `--dry-run` command shows copy-pasteable command that you can run
manually to see what breeze does. - then you can apply any modification to
those or try to mimic it
potiuk commented on PR #761:
URL: https://github.com/apache/airflow-site/pull/761#issuecomment-1507443433
> Followed 1. and 2. only. The script docs/build.sh does not exist anymore
in Airflow repository so I ran these commands instead to build the doc and
visualize them:
Look what
potiuk commented on PR #761:
URL: https://github.com/apache/airflow-site/pull/761#issuecomment-1505958273
This was always a bit of mistery to me especially when you make change to a
theme. But:
* Building and installing theme package locally
potiuk commented on PR #761:
URL: https://github.com/apache/airflow-site/pull/761#issuecomment-1503376025
My take:
* As long as we want to restrict indexing to "only" stable. Canonical is
good idea.
* We indeed need to manually update the existing link in all the past html
pages
potiuk commented on PR #761:
URL: https://github.com/apache/airflow-site/pull/761#issuecomment-1496593374
That's an interesting idea @mik-laj :) . It's not exactly why canonical was
created for, but it might do it in our case. However this is significantly more
complex to implement because
potiuk commented on PR #761:
URL: https://github.com/apache/airflow-site/pull/761#issuecomment-1496363194
> * Providers. Hard question. My personal opinion is we should come up with
a rule which applies to all providers. Having provider specific rules would be
too complex/time spending to
potiuk commented on PR #761:
URL: https://github.com/apache/airflow-site/pull/761#issuecomment-1496015676
(Hint): I am not brave enough :scream:
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to
potiuk commented on PR #761:
URL: https://github.com/apache/airflow-site/pull/761#issuecomment-1496014626
> Yeah I am happy with that. But we should add the big red banner with
warning that this is an old version and provide link for stable version.
Yeah would be nice if someone is
14 matches
Mail list logo