-
Notifications
You must be signed in to change notification settings - Fork 19
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
Version hasn't been being bumped when features were added. #13
Comments
I lost count. :P |
¯_(ツ)_/¯ sounds fine to me On Sun, Feb 15, 2015 at 12:21 PM, Joe DF notifications@github.com wrote:
|
How about moving code that is not fully working yet ((x)tea, ascii85?) to /src-dev/ and only keep properly working funcs in /src/ that way experiments in /src-dev/ don't require a bump in the version number. ❓ |
Anyways, branches are cheap. Maybe I should work on the installer in a separate branch as well? |
Hmmm... I don't know... What do you think @hi5 ? |
I don't mind if it is developed in the main branch - I haven't tried it yet but evilC has an installer script which may be of use https://github.com/evilC/AHK-Library-Setup |
Although that looks well thought out, the "put a placeholder #Include script in ProgFiles" is something I'd rather avoid for two reasons. The first is that it'd require administrator (which can be alleviated by putting it into the MyDocs folder), and the second is that it'd require the person to keep their copy of the library in the same spot all the time. This isn't an issue for people who clone repos into a central directory and leave it there, but that's not the average AHK user, I think. That being said, it made me realize one of the advantages of having a template. That is, when there's a template you can just #Include said template and use the changes live as they happen (without first requiring it to be built again). I think that make.ahk might want to also output a template to the build directory, for this purpose. We should consider opening another issue for further discussion. |
No description provided.
The text was updated successfully, but these errors were encountered: