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

Adding treepy to (Non)GNU ELPA #16

Open
monnier opened this issue Sep 15, 2024 · 2 comments
Open

Adding treepy to (Non)GNU ELPA #16

monnier opened this issue Sep 15, 2024 · 2 comments

Comments

@monnier
Copy link

monnier commented Sep 15, 2024

Any objection to adding this package to (Non)GNU ELPA?

@catern
Copy link

catern commented Sep 18, 2024

I'm also interested in this. I prefer NonGNU ELPA or MELPA-stable to just MELPA, and while treepy.el is in MELPA-stable, it's a fairly old version that has some issues.

@tarsius
Copy link
Contributor

tarsius commented Oct 1, 2024

I'm also interested in this. I prefer NonGNU ELPA or MELPA-stable to
just MELPA, and while treepy.el is in MELPA-stable, it's a fairly old
version that has some issues.

Once you have added this to NonGNU Elpa, you can remove it from MELPA, but you can also distribute it in both places. That's up to you.

If you do distribute it on both *elpa (and really even if you don't), you should make sure that the two sources of versioning truth agree with each other. I.e., when you create a new release bump the Version header and then tag that very commit with the corresponding tag. Currently this is broken:

                     *   651e263 (HEAD -> master) Merge pull request #15 from tarsiiformes/defun-indent
                     |\  
                     | * ee7bb91 (refs/pullreqs/15) No longer quote keywords
                     | * 635a690 Remove stray separator
                     | * 04ffce3 Use `defun' indentation type for treepy--context-assoc
                     | * f20bbeb Shorten long lines
                     |/  
                     *   75fe3ec Merge pull request #13 from tarsiiformes/typo
                     |\  
                     | * a0702bc (refs/pullreqs/13) Fix metastasized typo
                     |/  
                     *   298c49b Merge pull request #12 from cryptorick/bump-version
                     |\  
0.1.2 on NonGNU ELPA | * 59e573f (refs/pullreqs/12) bump version to 0.1.2
                     |/  
                     *   7c4a0d2 Merge pull request #11 from tarsiiformes/deprecate-testfn
                     |\  
                     | * ee815ce (refs/pullreqs/11) Deprecate use of map-contains-key's problematic TESTFN argument
                     |/  
                     *   de1a2dd Merge pull request #10 from tarsiiformes/unquote
                     |\  
                     | * 91829c8 (refs/pullreqs/10) Update library headers to follow conventions
                     | * 4f12d65 Enforce use of spaces for indentation
                     | * f653124 Don't quote keys in cl-case
                     |/  
0.1.2 on MELPA Stable* 3ac940e (tag: 0.1.2) Ignore `*.elc` files

(Or maybe NonGNU ELPA would consider 298c49b to be 0.1.2, I don't remember if it considers the first commit with a given version string to be that release, or the first such commit on the mainline. To avoid such uncertainties, I would recommend creating releases on the mainline, not on a release branch.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants