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.

Sign up for GitHub

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

Jump to bottom

Ensure manifest_path is read from config properly by Assembly #203

Merged

theodorton merged 1 commit into rails:main from keiththomps:fix-processor-generator

Sep 8, 2024

Contributor

keiththomps commented Sep 6, 2024

Ran into the following error attempting to utilize propshaft main (caused by simple typo):

Test likely isn't necessary to keep, but I needed it to reproduce the issue.


          Ensure manifest_path is read from config properly by Assembly

743c9d6

tapster commented Sep 7, 2024

Just hit the same issue. Confirmed Keith's fix in this PR resolves 👍 . thanks

2.943 bin/rails aborted!
2.945 NoMethodError: undefined method `manifest_path' for an instance of Pathname (NoMethodError)
2.945
2.945 load_path: load_path, output_path: config.output_path, compilers: compilers, manifest_path: config.manifest_path.manifest_path
2.945                                                                                                                  ^^^^^^^^^^^^^^

theodorton merged commit f1761f2 into rails:main

3 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet