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

[2_2_X][TIMOB-25372] Android: Generate metadata for Android support libraries #231

Merged
merged 2 commits into from Oct 17, 2017
Merged

[2_2_X][TIMOB-25372] Android: Generate metadata for Android support libraries #231

merged 2 commits into from Oct 17, 2017

Conversation

garymathews
Copy link
Contributor

@garymathews garymathews commented Oct 12, 2017

@janvennemann janvennemann changed the title [6_2_X][TIMOB-25372] Android: Generate metadata for Android support libraries [2_2_X][TIMOB-25372] Android: Generate metadata for Android support libraries Oct 17, 2017
Copy link
Contributor

@janvennemann janvennemann left a comment

Choose a reason for hiding this comment

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

Just some minor formatting note.

Please remove the changes related to the build as we should first check if that's easily possible. Preferably in a separate ticket. Thanks!

@@ -202,7 +202,8 @@ exports.cliVersion = '>=3.2';
function (next) {
var depMap = JSON.parse(fs.readFileSync(path.join(builder.platformPath, 'dependency.json')));
var libraryFilenames = depMap.libraries.appcompat;
libraryFilenames = libraryFilenames.concat(depMap.libraries.design || []);
libraryFilenames = libraryFilenames.concat(depMap.libraries.design || [], depMap.libraries.compat || [],
depMap.libraries.cardview || []);
Copy link
Contributor

Choose a reason for hiding this comment

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

I prefer to have each parameter on a separate line instead of only the last one and then indent it that much if the line gets too long.

libraryFilenames = libraryFilenames.concat(
  depMap.libraries.design || [],
  depMap.libraries.compat || [],
  depMap.libraries.cardview || []
);

@janvennemann janvennemann merged commit e97297f into tidev:2_2_X Oct 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants