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

Release hls-1.0.0 to hackage #1470

Closed
jneira opened this issue Mar 1, 2021 · 27 comments
Closed

Release hls-1.0.0 to hackage #1470

jneira opened this issue Mar 1, 2021 · 27 comments
Labels
CI Continuous integration old_type: meta Planing and organizing other issues

Comments

@jneira
Copy link
Member

jneira commented Mar 1, 2021

  • We have to start the process to upload all packages in this repo to hackage
  • We bumped out all bounds in the last release so we could upload their versions at https://github.com/haskell/haskell-language-server/tree/1.0.0, without creating a dedicated branch and backport .cabal changes (so it will be easier this time i hope)
    • imo we should not release other changes after 1.0.0
    • so f.e. we should release the hls-hlint-plugin version with the bug (1.0.0.0) before upload 1.0.0.1
  • The order would be: hls-plugin-api -> ghcide and co -> plugins -> hls itself

//cc @pepeiborra

@jneira jneira added CI Continuous integration old_type: meta Planing and organizing other issues labels Mar 2, 2021
@wz1000
Copy link
Collaborator

wz1000 commented Mar 2, 2021

Why do we need a hie-compat release? It hasn't changed at all.

@jneira
Copy link
Member Author

jneira commented Mar 3, 2021

In the last release we bumped up all packages versions to 1.0.0 and all deps bounds to 1.0.* for consistency, even if they did not change so the upload is needed

@wz1000
Copy link
Collaborator

wz1000 commented Mar 3, 2021

I don't think hie-compat needs to be at 1.0, I can revert that change if necessary. ghc-exactprintand hiedb were not bumped to 1.0 either, similarly hie-compat is more of a supporting library than a part of HLS.

@jneira
Copy link
Member Author

jneira commented Mar 3, 2021

Uploaded ghcide release candidate: https://hackage.haskell.org/package/ghcide-1.0.0.0/candidate

@Ailrun
Copy link
Member

Ailrun commented Mar 4, 2021

@jneira Oops, it still uses hie-compat >= 1.0.0.0

@jneira
Copy link
Member Author

jneira commented Mar 4, 2021

ugh did I forget to pull last changes? sure
will correct asap, thanks for check the candidate

@jneira
Copy link
Member Author

jneira commented Mar 4, 2021

@Ailrun corrected and uploaded again 🤞

@Ailrun
Copy link
Member

Ailrun commented Mar 4, 2021

Awesome, thanks!

@jneira
Copy link
Member Author

jneira commented Mar 4, 2021

ghcide fails due to the last version of haddock-library and the lack of upper bounds for it 😟

src\Development\IDE\Spans\Common.hs:123:28: error:
    • Couldn't match type ‘H.ModLink (H.DocH String String)’
                     with ‘[Char]’
      Expected type: String
        Actual type: H.ModLink (H.DocH String String)
    • In thehe first argument of ‘escapeBackticks’, namelymely ‘i’
      In the first argument of ‘(++)’, namelymely ‘escapeBackticks i’
      In the second argument of ‘(++)’, namelymely ‘escapeBackticks i ++ "`"’
    |
123 |   = "`" ++ escapeBackticks i ++ "`"
    |                            ^
cabal: Failed to build ghcide-0.6.0 (which is required by exe:ghcide-bench
from ghcide-0.6.0).

will open a pr for master and will backport in hls-1.0.0-hackage

@jneira
Copy link
Member Author

jneira commented Mar 4, 2021

Forgot to mention i did a hackage revision for ghcide-0.7.5, as it was failing for hls-plugin-api-1.0.0 and haddock-library: https://hackage.haskell.org/package/ghcide-0.7.5.0/revisions/

I assumed haddock-library was fixed in master

@jneira
Copy link
Member Author

jneira commented Mar 5, 2021

  • Pinging maintainers of each plugin: Release hls-1.0.0 to hackage #1470 (comment)
  • The hackage release should be done on the 1.0.0-hackage branch
    • Theorically, it has the sorrect versions and bounds but a sanity check building the plugin in isolation againts ghcide and hls-plugin-api 1.0.0.0, already released, would be nice
    • Possible changes in cabal files (no source files should be changed) should be done on master and backported to the branch (you can use mergifyio to backport changes auto)

@jneira
Copy link
Member Author

jneira commented Mar 5, 2021

@berberman maybe it would be handy if hls-haddock-comments-plugin could have some other hls maintainer(s). Feel free to add us, if anybody dont want to always can remove itself from the list 😄
@isovector same for hls-tacticswingman-plugin

@isovector
Copy link
Collaborator

@jneira I'd like to backport #1454, but don't know how to do that.

@jneira
Copy link
Member Author

jneira commented Mar 5, 2021

@isovector #1454 (comment) should work, i hope (good move @Ailrun 😄 )

it did! #1500

@isovector
Copy link
Collaborator

Thanks!

@berberman
Copy link
Collaborator

@jneira I uploaded hls-haddock-comments-plugin and added other maintainers.

@isovector
Copy link
Collaborator

isovector commented Mar 5, 2021

@jneira I've uploaded hls-tactics-plugin, and added @alanz @pepeiborra @Ailrun, @wz1000 and @jneira as maintainers.

@Ailrun
Copy link
Member

Ailrun commented Mar 5, 2021

hls-class-plugin is uploaded.

@pepeiborra
Copy link
Collaborator

Uploaded hls-explicit-imports-plugin and hls-retrie-plugin

@jneira
Copy link
Member Author

jneira commented Mar 6, 2021

Uploaded hls-hlint-plugin-1.0.0.1: https://hackage.haskell.org/package/hls-hlint-plugin-1.0.0.1

@jneira
Copy link
Member Author

jneira commented Mar 6, 2021

Uploaded hls-splice-plugin-1.0.0.0: https://hackage.haskell.org/package/hls-splice-plugin-1.0.0.0

@jneira
Copy link
Member Author

jneira commented Mar 6, 2021

@tittoassini hi! hls-eval-plugin is the last plugin required to upload hls itself
It would be great if you could add some other hls maintainer to the list of package maintainers, to agilize the release process 🙂

@tittoassini
Copy link
Contributor

tittoassini commented Mar 7, 2021 via email

@tittoassini
Copy link
Contributor

tittoassini commented Mar 7, 2021 via email

@jneira
Copy link
Member Author

jneira commented Mar 7, 2021

@tittoassini many thanks, the release is almost complete now

@jneira
Copy link
Member Author

jneira commented Mar 7, 2021

...and haskell-language-server-1.0.0.0 itself has been released after fixing a problem in its cabal file with 4c80cb3
will forward port to masterit already was corrected in master

@jneira jneira closed this as completed Mar 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous integration old_type: meta Planing and organizing other issues
Projects
None yet
Development

No branches or pull requests

7 participants