Skip to content

Conversation

@simoarpe
Copy link
Contributor

Resolves #20

Update to PSPDFKit 3.3.0

Signed-off-by: Simone Arpe <simon.arpe@gmail.com>
@simoarpe simoarpe self-assigned this Jul 26, 2017
@radazzouz
Copy link
Contributor

LGTM 👍

@radazzouz radazzouz self-requested a review July 26, 2017 15:08
@radazzouz
Copy link
Contributor

We could also bump the version to 1.3.1 (see #19) and tag as a new release.

simoarpe added 4 commits July 26, 2017 17:24
Signed-off-by: Simone Arpe <simon.arpe@gmail.com>
Signed-off-by: Simone Arpe <simon.arpe@gmail.com>
Signed-off-by: Simone Arpe <simon.arpe@gmail.com>
Signed-off-by: Simone Arpe <simon.arpe@gmail.com>
@juliangrosshauser
Copy link
Contributor

Does the iOS part still work after the react-native update?

@radazzouz
Copy link
Contributor

I stand by my initial approval! 👍

{
"name": "Catalog",
"version": "1.1.0",
"version": "1.2.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we keep the Catalog version in sync with the package version?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That would be ideally a good idea but highly impracticable. Consider the case when the Catalog needs to be upgraded and the package does not.
I would bump the corresponding version only when a change takes places, but we can discuss.

Copy link
Contributor

@juliangrosshauser juliangrosshauser left a comment

Choose a reason for hiding this comment

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

👍

@simoarpe simoarpe merged commit ce0b8d0 into master Jul 27, 2017
@simoarpe simoarpe deleted the simone/issue/20-update-to-3.3 branch July 27, 2017 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants