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

Add lookup_chain customizability #162

Merged
merged 1 commit into from
Jul 17, 2024

Conversation

joelzwarrington
Copy link
Contributor

Adds configuration to determine how components are found, so one wouldn't need to use the Component suffix in their components.

Added documentation to the README.md to configure this option

Copy link
Collaborator

@Spone Spone left a comment

Choose a reason for hiding this comment

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

👌

@joelzwarrington
Copy link
Contributor Author

hey @Spone is there anything I need to do to get this PR merged?

@Spone
Copy link
Collaborator

Spone commented Jul 11, 2024

I just want @nicolas-brousse to have a look before we merge this! Thanks for your patience :)

@nicolas-brousse
Copy link
Member

Hi @joelzwarrington looks good to me. Thank you for this PR and your patience!

Could you add a line in the Changelog and rebase from main. After that we could merge this :)

@kjellberg
Copy link

Need this 🙏🏻

@Spone Spone merged commit 087ed12 into pantographe:main Jul 17, 2024
60 checks passed
@joelzwarrington
Copy link
Contributor Author

hey @Spone / @nicolas-brousse do you have any idea when a new version will be released? Are there any other features you want included before that happens?

I'm currently using the git source in bundler but would be nice to use versions from https://rubygems.org

@nicolas-brousse
Copy link
Member

I didn't have time yesterday to do it after merging this. But I plan to do it today

@nicolas-brousse
Copy link
Member

@joelzwarrington 0.2.7 has just been released 😊

@joelzwarrington joelzwarrington deleted the optional-component-suffix branch July 18, 2024 15:51
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.

Make Component suffix optional in lookup
4 participants