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.

Sign up for GitHub

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

Jump to bottom

Delete all the things #2555

Merged
merged 2 commits into from
Nov 22, 2021
Merged

Delete all the things #2555

merged 2 commits into from
Nov 22, 2021

Conversation

nc6
Copy link
Contributor

@nc6 nc6 commented Nov 17, 2021

Well, not quite:

  • Remove various deprecated packages, modules, types and functions. There has now been ample time and downstream components have been updated.
  • Remove old symlinks to various packages.

nc6 added 2 commits November 17, 2021 11:06
@nc6
Remove symbolic links.
5ba0d75 
Dependencies have now had time to update.
@nc6
Remove deprecated packages, modules and functions.
451ae5d 
A little updating is needed, since it seems a few internal modules were
still using `ShelleyBased`.
@nc6 nc6 requested a review from lehins November 17, 2021 12:06
@JaredCorduan
Copy link
Contributor

beautiful!

lehins
lehins approved these changes Nov 18, 2021
View reviewed changes
Copy link
Collaborator

@lehins lehins left a comment

Choose a reason for hiding this comment

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

Hell yeah!

@nc6 nc6 merged commit 60e820d into master Nov 22, 2021
@iohk-bors iohk-bors bot deleted the nc/delete branch November 22, 2021 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lehins lehins lehins approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nc6 @JaredCorduan @lehins