-
Notifications
You must be signed in to change notification settings - Fork 21.4k
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
Convert CoffeeScript to ES6 syntax #38974
Conversation
@@ -15,8 +15,8 @@ module ActionCableHelper | |||
# Your CoffeeScript can then connect to the server without needing to specify the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs decaf here too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes!
Please squash to 1 commit 🙏 |
This PR allows for passing `if_exists` options to the `remove_index` method so that we can ignore already removed indexes. This work follows column `if/if_not_exists` from #38352 and `:if_not_exists` on `add_index` from #38555. We've found this useful at GitHub, there are migrations where we don't want to raise if an index was already removed. This will allow us to remove a monkey patch on `remove_index`. I considered raising after the `index_name_for_remove` method is called but that method will raise if the index doesn't exist before we get to execute. I have a commit that refactors this but after much consideration this change is cleaner and more straightforward than other ways of implementing this. This change also adds a little extra validation to the `add_index` test. Fix `nodoc` on edited methods.
Since 04cfbc8, a keepfile is generated in `tmp/pids/` to ensure that the directory always exists. However, the gitignore pattern for `/tmp/*` meant it wasn't tracked in git. To override that pattern we have to allow the `tmp/pids/` directory, ignore everything inside it, then finally allow `tmp/pids/.keep` again.
Follow up to 404e1a0.
Convert CoffeeScript to ES6 syntax
The rebase has pulled the last 5 commits into this branch as well. |
🤔 I think it should be enough to pull latest master (from upstream) and rebase you branch against master and force push. |
Whatever’s easier for you ✌️ |
Summary
Convert CoffeeScript to ES6 syntax for the example in the comments.