Skip to content
This repository has been archived by the owner on Jul 6, 2022. It is now read-only.

Release v5.0.1 #45

Merged
merged 7 commits into from
Nov 25, 2021
Merged

Release v5.0.1 #45

merged 7 commits into from
Nov 25, 2021

Conversation

inferrinizzard
Copy link
Owner

@inferrinizzard inferrinizzard commented Nov 25, 2021 •
edited
Loading

Patch for:

  • aliasAs option on demo page
  • AliasMode.select not working properly
  • * stripping whitespace when denseOperators is on

@inferrinizzard inferrinizzard added the release Label for Release PRs label Nov 25, 2021
@inferrinizzard inferrinizzard self-assigned this Nov 25, 2021
inferrinizzard
inferrinizzard commented Nov 25, 2021
View reviewed changes
src/core/Formatter.ts
@@ -219,7 +219,16 @@ export default class Formatter {
} else if (token.type === TokenType.RESERVED_LOGICAL_OPERATOR) {
formattedQuery = this.formatLogicalOperator(token, formattedQuery);
} else if (token.type === TokenType.RESERVED_KEYWORD) {
if (!(isToken.AS(token) && this.cfg.aliasAs === AliasMode.never)) {
if (
Copy link
Owner Author

Choose a reason for hiding this comment

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

add comments for cases

@inferrinizzard inferrinizzard merged commit 45d4dd2 into master Nov 25, 2021
inferrinizzard added a commit that referenced this pull request Dec 4, 2021
@inferrinizzard
v5.0.1 (#45)
35feb11 
* fix SELECT * with denseOperators

* fix aliasAs in test page

* fix AliasAs.select mode

* update README

* update issue templates

* update demo page on error

* add comments
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@inferrinizzard inferrinizzard

Labels
release Label for Release PRs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@inferrinizzard