Skip to content

Updated dependencies and added tests #53

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

Merged
merged 1 commit into from
Oct 16, 2020
Merged

Conversation

vtrifonov
Copy link
Contributor

@vtrifonov vtrifonov commented Oct 1, 2020

  • Updated bcel and common-io dependencies
  • Added test functionality for validating the tool output
    Related to Add tests #25

@cla-bot cla-bot bot added the cla: yes label Oct 1, 2020
@NathanaelA NathanaelA self-requested a review October 1, 2020 19:24
Copy link
Contributor

@NathanaelA NathanaelA left a comment

Choose a reason for hiding this comment

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

Hey, can you either reset the PR to use all these these JAR's/LIB'S that already exist in libs/
https://github.com/NativeScript/android-dts-generator/tree/master/libs/androidx/1.0.0
OR move them from libs/ to this location (and make sure whatever points to libs is updated). But we probably don't want TWO copies of most these in git...

@vtrifonov
Copy link
Contributor Author

sure, I can do this, but then if they are changed, the expected result should also be replaced

@vtrifonov vtrifonov force-pushed the trifonov/update-and-tests branch from d10631d to 45001ef Compare October 2, 2020 06:14
@vtrifonov vtrifonov requested a review from NathanaelA October 2, 2020 06:14
@vtrifonov
Copy link
Contributor Author

Done! Added also a note in the readme

@NathanaelA NathanaelA merged commit c94bde9 into master Oct 16, 2020
@NathanaelA NathanaelA deleted the trifonov/update-and-tests branch October 16, 2020 07:11
@NathanaelA

This comment was marked as abuse.

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.

2 participants