Skip to content

Conversation

@samWson
Copy link
Contributor

@samWson samWson commented Jan 23, 2021

The intention is to create the code supporting Exercism version 3 (V3) from a clean start. Refactoring
V2 code has proven too large a task.

With a V3 baseline we can load the code and dependencies into the image without any V2 code cluttering
things up. This also allows us to keep development of both versions in the same repository.

SourceImporter is just a class stub so that the baseline contains something and we can verify that it
loads correctly.

The V3 baseline can be loaded using Iceberg: In the repositories window, right click on the
pharo-smalltalk repository -> mouse over Metacello -> left click on Install baseline of Exercism...
-> enter v3 into the text field -> left click the OK button.

The intention is to create the code supporting Exercism version 3 (V3) from a clean start. Refactoring
V2 code has proven to large a task.

With a V3 baseline we can load the code and dependencies into the image without any V2 code cluttering
things up. This also allows us to keep development of both versions in the same repository.

`SourceImporter` is just a class stub so that the baseline contains something and we can verify that it
loads correctly.

The V3 baseline can be loaded using Iceberg:  In the repositories window, right click on the 
`pharo-smalltalk` repository -> mouse over `Metacello` -> left click on `Install baseline of Exercism...`
-> enter `v3` into the text field -> left click the `OK` button.
@samWson
Copy link
Contributor Author

samWson commented Jan 23, 2021

Hi @exercism/reviewers. Could I have an approval for this PR please?

This is some enabling work for #434. In Pharo Baselines are a tool for managing dependencies between packages in the image. They allow us to easily load groups of packages that are versioned by Git.

Having a V3 baseline makes it easy for me to load the packages separate from V2 while still using the same repository.

@SleeplessByte

This comment has been minimized.

@iHiD

This comment has been minimized.

@samWson samWson merged commit cc3f890 into exercism:master Jan 24, 2021
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