Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Feb 26, 2023. It is now read-only.

Update Kotlin example#2029

Merged
dodgex merged 2 commits into
androidannotations:developandroidannotations/androidannotations:developfrom
WonderCsabo:updateKotlinExampleWonderCsabo/androidannotations:updateKotlinExampleCopy head branch name to clipboard
Jul 30, 2017
Merged

Update Kotlin example#2029
dodgex merged 2 commits into
androidannotations:developandroidannotations/androidannotations:developfrom
WonderCsabo:updateKotlinExampleWonderCsabo/androidannotations:updateKotlinExampleCopy head branch name to clipboard

Conversation

@WonderCsabo

Copy link
Copy Markdown
Member

The manifest file is now found automatically, configuring it by hand
may cause issues.

@WonderCsabo WonderCsabo requested a review from dodgex July 26, 2017 11:54
@dodgex

dodgex commented Jul 26, 2017

Copy link
Copy Markdown
Member

Thank you, could you also update the versions in the other samples? To have them in sync.

@WonderCsabo

Copy link
Copy Markdown
Member Author

#2028 (comment)

Maybe because of this, we should not remove the kapt block from the example, because it fixes flavors with the stable Android Gradle plugin at least.

@dodgex

dodgex commented Jul 27, 2017

Copy link
Copy Markdown
Member

Maybe we should try to enhance the manifest finder to support the new(?) behaviour of kapt if they do not "fix" the issue you opened at kotlin.

I have an idea that could work, if i correctly understand the problem with the directories. I'll see if i get a hand on it this evening or tomorrow.

@WonderCsabo

Copy link
Copy Markdown
Member Author

@dodgex thanks!

@dodgex

dodgex commented Jul 27, 2017

Copy link
Copy Markdown
Member

PR is up

@WonderCsabo WonderCsabo force-pushed the updateKotlinExample branch from 45055d0 to 1e2e269 Compare July 30, 2017 09:18
The manifest file is now found automatically, configuring it by hand
may cause issues.
@WonderCsabo

Copy link
Copy Markdown
Member Author

@dodgex updated.

@dodgex

dodgex commented Jul 30, 2017

Copy link
Copy Markdown
Member

There is a travis issue due to the upgrade to 26 build tools. :/

@WonderCsabo WonderCsabo force-pushed the updateKotlinExample branch from 1e2e269 to 1cce92e Compare July 30, 2017 11:28
@WonderCsabo

Copy link
Copy Markdown
Member Author

@dodgex build fixed.

@dodgex dodgex added this to the 4.4 milestone Jul 30, 2017
@dodgex dodgex merged commit 1868ef6 into androidannotations:develop Jul 30, 2017
@WonderCsabo WonderCsabo deleted the updateKotlinExample branch February 16, 2019 21:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.