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

Add :fn/index as alias for :index in map syntax #608

Merged
merged 1 commit into from
Jul 17, 2024

Conversation

dgr
Copy link
Contributor

@dgr dgr commented Jul 17, 2024

This adds :fn/index as a preferred synonym for :index, which remains. The docs have been updated to discuss :fn/index and :index is not mentioned. If both :fn/index and :index are included in a query map, the value associated with :fn/index overrides the value associated with :index and :index is otherwise ignored.

Please complete and include the following checklist:

  • [x ] I have read CONTRIBUTING and the Etaoin Developer Guide.

  • [x ] This PR corresponds to an issue that the Etaoin maintainers have agreed to address.

  • [x ] This PR contains test(s) to protect against future regressions

  • [x ] I have updated CHANGELOG.adoc with a description of the addressed issue.

@lread lread merged commit 386918a into clj-commons:master Jul 17, 2024
53 checks passed
@dgr dgr deleted the dgr-index-to-fn branch July 24, 2024 20:49
lread added a commit that referenced this pull request Jul 28, 2024
…ridriver-logs

* origin/master:
  Ignore emacs backup files (#609)
  Add :fn/index as alias for :index in map syntax (#603) (#608)
  doc: thank Dave Roberts for contribution [skip ci] (#607)
  Add support for shadow DOM #604 (#606)
  doc: user-guide: add list of :fn/* (#605)
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