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

Update Fuel from upstream #14132

Merged

Conversation

theseion
Copy link
Contributor

  • remove FuelPlatform
  • properly store / load class comments and stamps
  • skip problematic tests when clean blocks are enabled
  • rename Fuel-Tests-Core to Fuel-Core-Tests

Fixes #14131

- remove FuelPlatform
- properly store / load class comments and stamps
- skip problematic tests when clean blocks are enabled
- rename Fuel-Tests-Core to Fuel-Core-Tests

Fixes pharo-project#14131
- remove dependency to FuelPlatform
- remove BaselineOfFuelPlatform
- fix depdendency on renamed Fuel-Core-Tests
@theseion
Copy link
Contributor Author

Fixed a lingering reference.

@theseion
Copy link
Contributor Author

Test run looks good to me.

@MarcusDenker
Copy link
Member

Hmm... lots of tests failing, e.g.

FLCreateTraitSerializationTest>>testCreateWithInternalTrait fais with: 'Attempt to read undeclared variable FLPlatform'

@MarcusDenker MarcusDenker added the Status: Need more work The issue is nearly ready. Waiting some last bits. label Jun 30, 2023
@theseion
Copy link
Contributor Author

theseion commented Jun 30, 2023 via email

@theseion
Copy link
Contributor Author

Looks good now, I think. There was some random error though.

@MarcusDenker MarcusDenker merged commit 666831c into pharo-project:Pharo12 Jul 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Need more work The issue is nearly ready. Waiting some last bits.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Fuel for Pharo 12 Cleanup: Fuel package(s) should follow regular package naming order
2 participants