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

Modify installation instructions to use import appsignal #201

Merged
merged 1 commit into from
Mar 19, 2024

Conversation

unflxw
Copy link
Contributor

@unflxw unflxw commented Mar 18, 2024

This is a sequel to #200 that I, uh, kinda forgot to do.


Modify installation instructions

Add a warning emoji whenever manual instrumentation is required,
that is, always.

For Django's instructions, refer to the modification needed as a
generic "your entrypoints" and leave it for the docs to explain
further.

Change other instructions to use import appsignal, replacing the
use of from __appsignal__ import appsignal.

Specify requests dependency in hook

I am not sure exactly why this change is needed now. I think Hatch
added support for specifying custom build hook dependencies
directly in the custom build hook in the last release, and this
might have accidentally broken the way in which we used to specify
this dependency.

It might also be that the breakage is intentional, in which case
they forgot to update the docs.

Add a warning emoji whenever manual instrumentation is required,
that is, always.

For Django's instructions, refer to the modification needed as a
generic "your entrypoints" and leave it for the docs to explain
further.

Change other instructions to use `import appsignal`, replacing the
use of `from __appsignal__ import appsignal`.
@unflxw unflxw added the chore label Mar 18, 2024
@unflxw unflxw self-assigned this Mar 18, 2024
@backlog-helper
Copy link

Hi @unflxw,

We've found some issues with your Pull Request.

  • This Pull Request does not include a changeset. Add a changeset if the change impacts users and should be included in the changelog upon release. Read more about changesets.
    Ignore this rule by adding [skip changeset] to your Pull Request body. - (More info)

New issue guide | Backlog management | Rules | Feedback

@unflxw unflxw force-pushed the modify-installation-appsignal-start branch from 7cf30c7 to 256b455 Compare March 19, 2024 09:36
@unflxw unflxw merged commit 232c037 into main Mar 19, 2024
12 of 18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants