Skip to content

Conversation

@Rinzwind
Copy link
Contributor

@Rinzwind Rinzwind commented Jul 8, 2025

This pull request updates the version in #iceberg for BaselineOfPharo for Pharo 12 to ‘v2.3.4’.

Before merging this please:

@tesonep
Copy link
Collaborator

tesonep commented Jul 11, 2025

Hi @Rinzwind, why in P13 we have ‘v2.3.2’ and in Pharo12 ‘v2.3.4’?

@Rinzwind
Copy link
Contributor Author

@tesonep The version number for Pharo 13 is ‘v2.4.2’, not ‘v2.3.2’. For both pull requests I just increased the last part of the version number: from 'v2.4.1' to 'v2.4.2' for Pharo 13 in pull request #18388, and from 'v2.3.3' to 'v2.3.4' for Pharo 12 in this pull request.

@Rinzwind
Copy link
Contributor Author

@tesonep Could you maybe merge this? If you can first merge the two pull requests this depends on at least: libgit2-pharo-bindings pull request #101 and Iceberg pull request #1949.

@Rinzwind Rinzwind closed this Jul 23, 2025
@Rinzwind Rinzwind reopened this Jul 23, 2025
@Rinzwind Rinzwind closed this Jul 23, 2025
@Rinzwind Rinzwind reopened this Jul 23, 2025
@Rinzwind
Copy link
Contributor Author

The tests that failed in build 3 for this pull request also failed in build 1575 of the ‘Pharo12’ job.

@Rinzwind
Copy link
Contributor Author

@Ducasse @jecisc Thanks again for giving me the necessary permissions, I was able to merge all the pull requests and create the tags and releases needed for merging this pull request and pull request #18388.

Rinzwind added a commit to Rinzwind/pharo that referenced this pull request Jul 24, 2025
…pharo12-iceberg

Update version in #iceberg for BaselineOfPharo for Pharo 12 to ‘v2.3.4’
@Rinzwind
Copy link
Contributor Author

I don’t have permission to merge this though, perhaps because of the failing tests, as those are not related as far as I can tell, I tried directly pushing merge commits for this pull request and pull request #18388, but that didn’t work either.

@tesonep @Ducasse @jecisc Could one of you maybe merge pull request #18388 and this one?

@jecisc jecisc merged commit a8c5eb3 into pharo-project:Pharo12 Jul 24, 2025
1 check failed
@Rinzwind
Copy link
Contributor Author

@jecisc Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants