-
Notifications
You must be signed in to change notification settings - Fork 515
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
The Falcon integration for Falcon 3.0.1 #1297
The Falcon integration for Falcon 3.0.1 #1297
Conversation
This pull request has gone three weeks without activity. In another week, I will close it. But! If you comment or otherwise update it, I will reset the clock, and if you label it "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
any news about this ? |
Hello @bartolootrit ! We also got an PR for Falcon 3.0.0 from @Seluj78: Could you @bartolootrit talk to @Seluj78 on how you can merge the two PRs into one? So we have only one Falcon Integration that we can review and eventually merge? Thanks! |
Maybe also @pecalleja could help somehow? |
This pull request has gone three weeks without activity. In another week, I will close it. But! If you comment or otherwise update it, I will reset the clock, and if you label it "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
#1073 was closed. Are my changes from there in here ? :) |
Hey @Seluj78 |
FYI @bartolootrit and @Seluj78 I have now added Falcon 3.0.0 and 3.1.0 to our test suite here: If all the tests for this are green I can merge this into master and then you can rebase your pull request onto the new master, so we can run all our tests in Falcon 3.0 and 3.1! |
Looking good, it just seems like the imports are differently named now ? https://github.com/getsentry/sentry-python/pull/1073/files#diff-af8d1eba8feb85336f0d9a1a04a78beaa87da4a8efa675b478a18b8f603d63f8R20 |
@antonpirker |
Hey @bartolootrit ! Turns out I can not merge #1378 because it fails (because it has not Falcon 3 support) :-) So I changed this PR to be merge into the branch with the Tests. Lets see how this turns out. |
@antonpirker I've rebased this branch. An approval is needed to run workflows. |
The doc still say that it is compatible only with 1.4 and 2.0 only Could you please confirm that it is now compatible with falcon 3.0.1 ? |
Is there any reason why this pull requests has never been done ? |
Why this pull request has never been merged ? Could be great to have falcon 3 support ! |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Sorry for the huuuge delay on this. I really messed this up. I have now merged this into another branch that has updates to the test matrix for Falcon: |
The Falcon integration passed the tests for Falcon 3.0.1, 2.0.0, 1.4.0 (Python 3.8) and Falcon 2.0.0, 1.4.0 (Python 2.7)