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

Fix Dash destructuring. Add tests. Use Github Actions. #102

Merged
merged 2 commits into from
Oct 20, 2021
Merged

Conversation

okamsn
Copy link
Owner

@okamsn okamsn commented Oct 20, 2021

Fixes #101.

  • Instead of trying to swap out variables in the variable list, just check for
    variables made by dash--match-make-source-symbol. This create more
    variables than needed, but should be more reliable.
  • Add more tests to destructure using quoted symbols as keys, such as in
    (&alist 'a 'b). Previously, these were mistaken as lists like (quote a),
    which would be further destructured.
  • Use GitHub Actions to automatically run the tests for Dash compatibility.
  • Update the required version of Dash from "2" to "2.19".

- Correctly handle quoted symbols in `loopy-dash` variable list.
  We were incorrectly destructuring forms of `(quote symbol)` as a list.
- Update Dash requirement to version 2.19.
- Add more tests for Dash.
- Use Github Actions to run the Dash tests.
- Update `CHANGELOG.md`.
@okamsn okamsn merged commit 0fbd17b into master Oct 20, 2021
@okamsn okamsn deleted the fix-dash branch March 30, 2022 03:01
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.

Fix Dash destructuring
1 participant