Skip to content

remove jstin from important packages #17765

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Apr 18, 2021
Merged

remove jstin from important packages #17765

merged 1 commit into from
Apr 18, 2021

Conversation

ringabout
Copy link
Member

@ringabout ringabout commented Apr 18, 2021

Ref #17759
Ref #17711

https://github.com/LemonBoy/jstin/blob/master/src/jstin.nim is already archived and cannot pass CI anymore.

@ringabout
Copy link
Member Author

How about putting the source code and tests to Nim repo? I remember the code caught a regression ever.

@Araq Araq merged commit ca3fe63 into devel Apr 18, 2021
@Araq Araq deleted the xflywind-patch-3 branch April 18, 2021 13:30
@Araq
Copy link
Member

Araq commented Apr 18, 2021

How about putting the source code and tests to Nim repo? I remember the code caught a regression ever.

If you feel like it.

@timotheecour
Copy link
Member

  • @LemonBoy I don't see any note or issue report mentioning the reason for archival, please comment if you can

How about putting the source code and tests to Nim repo? I remember the code caught a regression ever.

let's fork it instead, eg under nim-lang/nim, and change nimble url to point to the new fork

@ringabout
Copy link
Member Author

please do it, i don't have the right.

@timotheecour
Copy link
Member

timotheecour commented Apr 18, 2021

done => https://github.com/nim-lang/jstin

i don't have the right.

@Araq how about giving @xflywind those rights? there were a few other instances like that

TODO:

  • make a PR against nimble to update the ref
  • change ref in important_packages
  • add commit to fix the jstin CI error re default (remove default jstin#1)

@timotheecour timotheecour added the TODO: followup needed remove tag once fixed or tracked elsewhere label Apr 18, 2021
@ringabout ringabout removed the TODO: followup needed remove tag once fixed or tracked elsewhere label Apr 25, 2021
ringabout added a commit that referenced this pull request Apr 25, 2021
PMunch pushed a commit to PMunch/Nim that referenced this pull request Mar 28, 2022
PMunch pushed a commit to PMunch/Nim that referenced this pull request Mar 28, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants