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

hooks with point: return are not triggered #36

Open
jmettraux opened this issue Nov 27, 2020 · 2 comments
Open

hooks with point: return are not triggered #36

jmettraux opened this issue Nov 27, 2020 · 2 comments
Assignees
Labels

Comments

@jmettraux
Copy link
Member

As seen with @northox and @jfrioux https://gitter.im/floraison/flor?at=5fbfb55f846eed6b67ae2bf2

Reproduced in https://github.com/jmettraux/issue_flor_hook_20201126

It seems hooks with point: return are not triggered.

The workaround is to write point: "return" or point: 'return'.

[
  { point: execute, consumed: true, nid: '0', require: 'pollen/pollen.rb', class: 'LaunchedPollenHook' },
  { point: return, consumed: true, require: 'pollen/pollen.rb', class: 'ReturnedPollenHook' },
  { point: terminated, consumed: true, require: 'pollen/pollen.rb', class: 'TerminatedPollenHook' },
  { point: failed, consumed: true, require: 'pollen/pollen.rb', class: 'FailedPollenHook' },
]

Probably the point gets evaluated to an empty string (and never matches).

@jmettraux jmettraux added the bug label Nov 27, 2020
@jmettraux jmettraux self-assigned this Nov 27, 2020
@jmettraux
Copy link
Member Author

@northox @jfrioux I think the commit above solves it. Please tell me if not. Thanks again for reporting that!

@jmettraux
Copy link
Member Author

@jfrioux wrote in floraison/pollen#5 :

Though, just tested with commit 465d4ed, on Ruby 2.3.8 and 2.6.6 and the issue remains when return is unquoted.

Reopening...

@jmettraux jmettraux reopened this Dec 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant