Skip to content

Fix up tests#4

Merged
doublep merged 3 commits into
flycheck:masterfrom
xendk:fix-tests
Aug 7, 2025
Merged

Fix up tests#4
doublep merged 3 commits into
flycheck:masterfrom
xendk:fix-tests

Conversation

@xendk

@xendk xendk commented Aug 3, 2025

Copy link
Copy Markdown
Contributor

Add lexical-binding and newer Emacs versions.

@xendk

xendk commented Aug 3, 2025

Copy link
Copy Markdown
Contributor Author

Let's see what the runners says to this.

@xendk

xendk commented Aug 3, 2025

Copy link
Copy Markdown
Contributor Author

Well, apparently newest flycheck requires Emacs 27, so remove the older versions.

Now it's only the snapshot release that fails.

@doublep

doublep commented Aug 3, 2025

Copy link
Copy Markdown
Member

Do you see what's wrong on the snapshot?

@xendk

xendk commented Aug 3, 2025

Copy link
Copy Markdown
Contributor Author

Well, it seems that it can't find dependency-a when checking test/project-a.el.

test/project-a.el requires test-util.el which in turn requires project-a which is the one with the dependency-a dependency.

Could it be that the load-path isn't set up properly with package archives when running on non-main files? I must admit this is a bit magic to me at the moment.

@doublep doublep merged commit d2c9c6b into flycheck:master Aug 7, 2025
11 of 12 checks passed
@doublep

doublep commented Aug 7, 2025

Copy link
Copy Markdown
Member

OK, I will investigate when I found time.

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.

2 participants