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

Finalize targets #2684

Merged
merged 5 commits into from Aug 28, 2017

Conversation

Projects
None yet
2 participants
@matthid
Member

matthid commented Aug 28, 2017

  • Bundle the correct targets file
  • Be compatible with the old targets file
  • *.references are now called *.paket.resolved in the obj directory

matthid added some commits Aug 28, 2017

Fixes various isses with the alpha release
- Bundle the correct targets file
- Be compatible with the old targets file
- *.references are now called *.paket.resolved in the obj directory
@matthid

This comment has been minimized.

Show comment
Hide comment
@matthid

matthid Aug 28, 2017

Member

@alfonsogarciacaro I renamed the files to *.paket.resolved instead of *.paket.references as that probably better matches their content. Because of all the issues with the alpha I'll do another beta before releasing this. Thanks for the testing!

Member

matthid commented Aug 28, 2017

@alfonsogarciacaro I renamed the files to *.paket.resolved instead of *.paket.references as that probably better matches their content. Because of all the issues with the alpha I'll do another beta before releasing this. Thanks for the testing!

@alfonsogarciacaro

This comment has been minimized.

Show comment
Hide comment
@alfonsogarciacaro

alfonsogarciacaro Aug 28, 2017

@matthid Ok, thanks! But please don't change the name again after Fable 1.2 RTM is out ;)

alfonsogarciacaro commented Aug 28, 2017

@matthid Ok, thanks! But please don't change the name again after Fable 1.2 RTM is out ;)

@matthid

This comment has been minimized.

Show comment
Hide comment
@matthid

matthid Aug 28, 2017

Member

@alfonsogarciacaro Sure :) We don't try to break stuff we released but we didn't release the other name-change yet ;)

Member

matthid commented Aug 28, 2017

@alfonsogarciacaro Sure :) We don't try to break stuff we released but we didn't release the other name-change yet ;)

@matthid

This comment has been minimized.

Show comment
Hide comment
@matthid

matthid Aug 28, 2017

Member

Tbh I didn't even expected fable to pick that up soooo fast 👍 But in the end it turned out to be good because we found some bugs before releasing them to the wild :)

Member

matthid commented Aug 28, 2017

Tbh I didn't even expected fable to pick that up soooo fast 👍 But in the end it turned out to be good because we found some bugs before releasing them to the wild :)

@matthid matthid merged commit c3a8cc5 into master Aug 28, 2017

0 of 4 checks passed

continuous-integration/appveyor/branch Waiting for AppVeyor build to complete
Details
continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment