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

Removed the from_display_name function from Item #15

Merged
merged 2 commits into from
Feb 26, 2021
Merged

Removed the from_display_name function from Item #15

merged 2 commits into from
Feb 26, 2021

Conversation

kallekankaanpaa
Copy link
Contributor

@kallekankaanpaa kallekankaanpaa commented Feb 26, 2021

  • Changed the Item generator to not generate from_display_name
    function because of it's abiguos
  • Updated libcraft-generators README to include all currently
    implemented generators

closes #14

- Changed the `Item` generator to not generate `from_display_name`
  function because of it's abiguos
- Updated `libcraft-generators` README to include all currently
  implemented generators
@kallekankaanpaa kallekankaanpaa changed the title Removed the from_display_name fuction from Item Removed the from_display_name fuction from Item (#14) Feb 26, 2021
@kallekankaanpaa kallekankaanpaa changed the title Removed the from_display_name fuction from Item (#14) Removed the from_display_name fuction from Item Feb 26, 2021
@kallekankaanpaa kallekankaanpaa changed the title Removed the from_display_name fuction from Item Removed the from_display_name function from Item Feb 26, 2021
Copy link
Member

@caelunshun caelunshun left a comment

Choose a reason for hiding this comment

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

Thanks! Not sure why I initially added from_display_name, but it's not useful anyway.

@caelunshun caelunshun merged commit 9f7eadb into feather-rs:master Feb 26, 2021
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

Successfully merging this pull request may close these issues.

Multiple items with same display name.
3 participants