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

Fix hardcoded version number #202

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

smartlitchi
Copy link

@smartlitchi smartlitchi commented Oct 26, 2020

This pull request fixes #200.

@smartlitchi smartlitchi changed the title Fix hardcoded version number (#200) Fix hardcoded version number Oct 26, 2020
@moviuro
Copy link
Contributor

moviuro commented Oct 26, 2020

Wouldn't that break in production? git isn't a dependency of rofi-pass; and a version number should be hardcoded in packaged versions.

@smartlitchi smartlitchi marked this pull request as draft October 26, 2020 15:32
@smartlitchi
Copy link
Author

You are absolutely right. I haven't thought about that. It's one of my first pull requests so I still lack experience. I will try to find a better way

@smartlitchi smartlitchi marked this pull request as ready for review October 28, 2020 09:00
@smartlitchi
Copy link
Author

I couldn't find a simple way to get the version number without hardcoding it so at least I changed the version number to the correct one.

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.

Version is still 1.5.3
2 participants