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

Fixing baseline by removing Espell and SpecDebugger dependencies #4

Merged
merged 3 commits into from
Sep 13, 2022

Conversation

adri09070
Copy link
Contributor

@adri09070 adri09070 commented Sep 9, 2022

Fix #3

Fixing baseline by removing deprecated dependencies to Espell and SpecDebugger.

To do that, I also removed all classes that use the deprecated SpecDebugger UI or Espell as an AST evaluator isn't really needed

spec for: #common do: [ "Dependencies"
spec
baseline: 'Ghost'
with: [ spec repository: 'gitlab://gitlab.inria.fr:RMOD/Ghost' ].
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is Ghost really needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't know. I just left it because it is needed for an "experimental" package that I will never load anyway but I can delete the dependency and the package if you want

Copy link
Contributor Author

@adri09070 adri09070 Sep 12, 2022

Choose a reason for hiding this comment

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

I removed Ghost and the overlay from the baseline. In this case, it would also fix #1

@guillep guillep merged commit 7707a90 into RMODINRIA:master Sep 13, 2022
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.

Some dependencies are missing in the baseline
2 participants