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

[WIP] Core Renderer #580

Draft
wants to merge 3 commits into
base: v3_release__vue3_rails7_gem_split
Choose a base branch
from

Conversation

fiedl
Copy link
Collaborator

@fiedl fiedl commented Feb 1, 2022

work in progress

- Show the matestack backtrace, which is hidden by default by the backtrace silencer, in the terminal
- Remove the docker directory `/app/` from the beginning of the paths; this allows, for example, to use iTerm's semantic-history feature, which is useful for debugging

https://api.rubyonrails.org/classes/ActiveSupport/BacktraceCleaner.html
https://github.com/rails/rails/blob/main/activesupport/lib/active_support/backtrace_cleaner.rb
@fiedl fiedl changed the base branch from main to v3_release__vue3_rails7_gem_split February 1, 2022 22:26
@fiedl fiedl marked this pull request as draft February 1, 2022 22:30
in order to allow `binding.pry` in development

https://github.com/pry/pry
https://github.com/pry/pry-rails
https://github.com/deivid-rodriguez/pry-byebug

Hint: When using this with docker, you need to attach to the rails-server docker container to use the interactive pry session:

    docker-compose up dummy
    docker attach matestack-ui-core_dummy_1
`pry` requires a newer version of `less` to display long results properly.

pry/pry#1248
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant