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

Remove p shorthand deprecated in #39 #40

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

kaspth
Copy link
Contributor

@kaspth kaspth commented Jul 3, 2022

Once we've shipped the deprecation, we can merge this in later to remove p entirely.

@kaspth
Copy link
Contributor Author

kaspth commented Jul 3, 2022

CI's going to fail because the depending libraries haven't removed p calls yet. Expected.

@kaspth
Copy link
Contributor Author

kaspth commented Jul 3, 2022

We can rerun the CI on this to check that we've removed all the p calls from depending libraries too.

Base automatically changed from remove-partial-shorthands to features/partial-instead-of-p July 4, 2022 13:47
Base automatically changed from features/partial-instead-of-p to main July 4, 2022 14:14
@andrewculver
Copy link
Contributor

@kaspth Want to take a stab at resolving the conflicts?

@kaspth kaspth force-pushed the remove-p-shorthand-post-deprecation branch from 60d4ac2 to 00bf2c1 Compare July 27, 2022 12:51
@kaspth
Copy link
Contributor Author

kaspth commented Jul 27, 2022

@andrewculver got it resolved here

@kaspth kaspth added this to the v1.0.0 milestone Aug 29, 2022
* main:
  Ignore /tmp if you're running BT in tmp/starter like CI does
  Fix some CHANGELOG issues and refit some examples + words
  Skip needless CI steps (#60)
  We don't need to depend on sqlite3 that was for an earlier version of the ViewComponent integration
  Fix #54 to use new section yield syntax
  Mixing `yield` call styles (#54)
  Expose Sections as an alternative to `content_for` (#57)
  Introduce Capybara to the test suite (#55)
  Add `content_from` to let partials relay contents (#53)
  v0.1.9
  v0.1.8
  Document release 🎉 (#52)
  Rename `Partial#output_buffer` to `Partial#yield` (#41)
  Allow outer partial access when capturing (#49)
  Fix accessing `partial` before rendering leaks state (#47)
  Use load hooks for monkey_patch loading (#48)
@kaspth kaspth mentioned this pull request Nov 7, 2022
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