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

Change binder link from master to binder branch for robustness #2703

Closed

Conversation

MarcSkovMadsen
Copy link
Collaborator

@MarcSkovMadsen MarcSkovMadsen commented Sep 2, 2021

Addresses suggestion 2 of #2702

The master branch is always changing which means

  • its not consistent with the docs for the current release
  • Starting Binder takes a long time because it has to build a new image
  • might not work because the master branch does not work.

I suggest changing to a branch called binder.

I've already created the binder branch from the v0.12.1 (git checkout tags/v0.12.1) and tested that it works.

You can test binder via https://mybinder.org/v2/gh/holoviz/panel/binder?urlpath=lab/tree/examples

Release Procedure

Please note that when ever a new version of Panel is released a step in that process is to merge the release into the binder branch and manually test that binder can build the image and that things works. But it would be a good test anyways that Panel indeed works in a fresh environment and that all the gallery apps works etc.

Improvements to binder branch

Any improvements to the binder branch during releases should start from the binder branch and not from master.

@codecov
Copy link

codecov bot commented Sep 2, 2021

Codecov Report

Merging #2703 (0c848ae) into master (3b6f0c9) will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2703      +/-   ##
==========================================
- Coverage   81.81%   81.81%   -0.01%     
==========================================
  Files         187      187              
  Lines       23905    23905              
==========================================
- Hits        19558    19557       -1     
- Misses       4347     4348       +1     
Impacted Files Coverage Δ
panel/tests/pane/test_plotly.py 98.10% <0.00%> (-0.64%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3b6f0c9...0c848ae. Read the comment docs.

@philippjfr
Copy link
Member

Shouldn't we point it to the last release instead?

@@ -11,7 +11,7 @@
| Latest dev release | [![Github tag](https://img.shields.io/github/v/tag/holoviz/panel.svg?label=tag&colorB=11ccbb)](https://github.com/holoviz/panel/tags) [![dev-site](https://img.shields.io/website-up-down-green-red/https/pyviz-dev.github.io/panel.svg?label=dev%20website)](https://pyviz-dev.github.io/panel/) |
| Latest release | [![Github release](https://img.shields.io/github/release/holoviz/panel.svg?label=tag&colorB=11ccbb)](https://github.com/holoviz/panel/releases) [![PyPI version](https://img.shields.io/pypi/v/panel.svg?colorB=cc77dd)](https://pypi.python.org/pypi/panel) [![panel version](https://img.shields.io/conda/v/pyviz/panel.svg?colorB=4488ff&style=flat)](https://anaconda.org/pyviz/panel) [![conda-forge version](https://img.shields.io/conda/v/conda-forge/panel.svg?label=conda%7Cconda-forge&colorB=4488ff)](https://anaconda.org/conda-forge/panel) [![defaults version](https://img.shields.io/conda/v/anaconda/panel.svg?label=conda%7Cdefaults&style=flat&colorB=4488ff)](https://anaconda.org/anaconda/panel) |
| Docs | [![gh-pages](https://img.shields.io/github/last-commit/holoviz/panel/gh-pages.svg)](https://github.com/holoviz/panel/tree/gh-pages) [![site](https://img.shields.io/website-up-down-green-red/https/panel.holoviz.org.svg)](https://panel.holoviz.org) |
| Binder | [![Binder](https://mybinder.org/badge_logo.svg)](https://mybinder.org/v2/gh/holoviz/panel/master?urlpath=lab/tree/examples) |
| Binder | [![Binder](https://mybinder.org/badge_logo.svg)](https://mybinder.org/v2/gh/holoviz/panel/binder?urlpath=lab/tree/examples) |
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
| Binder | [![Binder](https://mybinder.org/badge_logo.svg)](https://mybinder.org/v2/gh/holoviz/panel/binder?urlpath=lab/tree/examples) |
| Binder | [![Binder](https://mybinder.org/badge_logo.svg)](https://mybinder.org/v2/gh/holoviz/panel/v0.12.1?urlpath=lab/tree/examples) |

@philippjfr philippjfr closed this Sep 2, 2021
@philippjfr philippjfr deleted the change-binder-link-from-master-to-binder branch September 20, 2021 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants