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

Move to Iceberg v2 #8461

Merged
merged 9 commits into from
Feb 9, 2021
Merged

Conversation

tinchodias
Copy link
Contributor

@Ducasse
Copy link
Member

Ducasse commented Jan 31, 2021

Hi martin

The build says:
NewUndeclaredWarning: IceTipNewRepositoryPanel>>newRepository (IceRepositoryCreator is Undeclared)
NewUndeclaredWarning: IceTipRepositoryTypePanel>>defaultLocation (IceLibgitRepository is Undeclared)
NewUndeclaredWarning: IceTipRepositoryTypePanel>>defaultSubdirectory (IceLibgitRepository is Undeclared)
MetacelloNotification: Loaded -> Iceberg-TipUI-tonel.1 --- github://pharo-vcs/iceberg:v2.0.0 --- cache
MetacelloNotification: Loading -> Iceberg-Plugin-tonel.1 --- github://pharo-vcs/iceberg:v2.0.0 --- cache
MetacelloNotification: Loaded -> Iceberg-Plugin-tonel.1 --- github://pharo-vcs/iceberg:v2.0.0 --- cache
MetacelloNotification: Loading -> Iceberg-Plugin-Calypso-tonel.1 --- github://pharo-vcs/iceberg:v2.0.0 --- cache
Error: This package depends on the following classes:
  ClyIcebergShowMethodVersionCommand
You must resolve these dependencies before you will be able to load these definitions: 
  ClyIcebergShowMethodVersionCommand>>#execute

MCPackageLoader(Object)>>notify:
MCPackageLoader>>warnAboutDependencies
MCPackageLoader>>validate
MCPackageLoader>>loadWithNameLike:
MCVersionLoader>>basicLoadWithNameLike:
[ self basicLoadWithNameLike: aString ] in MCVersionLoader>>loadWithNameLike: in Block: [ self basicLoadWithNameLike: aString ]
[ returnValue := aBlock value ] in [ 
		[ returnValue := aBlock value ] ensure: [ 
			self announceLoadStop: aString ] ] in MCVersionLoader>>announceLoad:do: in Block: [ returnValue := aBlock value ]
BlockClosure>>ensure:
[ 

tinchodias added a commit to pharo-vcs/iceberg that referenced this pull request Feb 1, 2021
Remove extension to Iceberg-Plugin-Calypso... It's fixed in other way in
pharo-project/pharo#8461
@tinchodias
Copy link
Contributor Author

What are the yellow "Recording test results [Checks API] No suitable checks publisher found."?

@MarcusDenker
Copy link
Member

Four tests are failing:

testEqualMethodNeedsToBeInComparingProtocol

Violations: IceTipCachedModel>>#=.   Reason: IceTipCachedModel>>#= subclasses should have #Object in protocol =

testMethodsContainNoHalt

Found methods with haltan OrderedCollection(IceFileSystemDefinition>>#path:)

testExternalIDEDependencies

Given Collections do not match!  additions : #(#'Ring-Core')  missing: #()

testDefaultImageDoesNotStoreAnySetting

Given Collections do not match!  additions : an Array(a StoredSetting('#commitPreferences#saveImageOnCommit') a StoredSetting('#commitPreferences#pushOnCommit'))  missing: #()

@MarcusDenker MarcusDenker added the Status: Need more work The issue is nearly ready. Waiting some last bits. label Feb 2, 2021
@tinchodias
Copy link
Contributor Author

tinchodias commented Feb 2, 2021

Thank you, Marcus. I'd like to learn where to click to see the errrors. After clicking in the "continuous-integration/jenkins/pr-merge" Details (https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%20and%20branch%20Pipeline/job/PR-8461/4/display/redirect)

I clicked on unix-64 which had the yellow warning mark, and then clicked to expand each of the horizontal small reports below... and couldn't see that info.

@tinchodias
Copy link
Contributor Author

I have fixed 2/4. Remaining:

In Iceberg/dev-2.0 branch they were placed
in another package to ease
development.
tinchodias added a commit to pharo-vcs/iceberg that referenced this pull request Feb 4, 2021
tinchodias added a commit to pharo-vcs/iceberg that referenced this pull request Feb 4, 2021
@tinchodias
Copy link
Contributor Author

I didn't understand where are the failed tests, with the new commit.

Anyway I wanted to wait until tomorrow for a changeset from "gcorriga" user in Discord, to update the world menu entry.

@tinchodias
Copy link
Contributor Author

I managed to read the output of Jenkins. I'm fixing the failing dependency test.

tinchodias added a commit to pharo-vcs/iceberg that referenced this pull request Feb 8, 2021
It was composed of 3 method
extensions that I'm moving to
Pharo 9 (see:
pharo-project/pharo#8461)
@tinchodias tinchodias changed the title Move to iceberg:v2.0.0 Move to Iceberg v2 Feb 8, 2021
@tinchodias
Copy link
Contributor Author

This time, only windows build failed

@MarcusDenker MarcusDenker added Status: Tests passed please review! and removed Status: Need more work The issue is nearly ready. Waiting some last bits. labels Feb 8, 2021
@tesonep tesonep merged commit 40984a1 into pharo-project:Pharo9.0 Feb 9, 2021
@tinchodias tinchodias deleted the merge-iceberg-2-0-0 branch February 11, 2021 15:37
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.

unused Ivars in Iceberag: IceTipAdoptCommitCommand , LGitReadStream
4 participants