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.

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

feat: copilot frontend e2e #7694

Merged
merged 1 commit into from
Oct 29, 2024
Merged

feat: copilot frontend e2e #7694

merged 1 commit into from
Oct 29, 2024

Conversation

darkskygit
Copy link
Member

@darkskygit darkskygit commented Aug 1, 2024 •
edited
Loading

fix AF-1432

@graphite-app Graphite App
Copy link

graphite-app bot commented Aug 1, 2024

Your org has enabled the Graphite merge queue for merging into canary

Add the label “merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

@darkskygit Graphite App
Copy link
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @darkskygit and the rest of your teammates on Graphite Graphite

@github-actions github-actions bot added test Related to test cases app:core labels Aug 1, 2024
@nx-cloud Nx Cloud
Copy link

nx-cloud bot commented Aug 1, 2024 •
edited
Loading

☁️ Nx Cloud Report

CI is running/has finished running commands for commit d718527. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution

✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

@codecov Codecov
Copy link

codecov bot commented Aug 1, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.18%. Comparing base (5709ebb) to head (d718527).
Report is 1 commits behind head on canary.

Additional details and impacted files 
@@           Coverage Diff           @@
##           canary    #7694   +/-   ##
=======================================
  Coverage   70.18%   70.18%           
=======================================
  Files         536      536           
  Lines       33717    33717           
  Branches     3023     3023           
=======================================
  Hits        23665    23665           
  Misses       9697     9697           
  Partials      355      355
Flag Coverage Δ
server-test 77.09% <100.00%> (ø)
unittest 45.31% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@darkskygit darkskygit force-pushed the darksky/frontend-copilot-e2e branch 2 times, most recently from f82dcd4 to 10c7cde Compare August 5, 2024 08:24
@darkskygit darkskygit marked this pull request as ready for review August 5, 2024 08:36
pengx17
pengx17 reviewed Aug 5, 2024
View reviewed changes
Copy link
Collaborator

@pengx17 pengx17 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how much does it cost when running one run of the ai tests?

@darkskygit darkskygit force-pushed the darksky/frontend-copilot-e2e branch 2 times, most recently from ee45421 to de693ef Compare August 5, 2024 08:56
@darkskygit
Copy link
Member Author

how much does it cost when running one run of the ai tests?

with the current number of use cases, a complete test make 19 requests, and the required amount is less than $0.01
image

@darkskygit darkskygit force-pushed the darksky/frontend-copilot-e2e branch 2 times, most recently from 0d59b04 to 8ee58a5 Compare August 5, 2024 10:36
@darkskygit darkskygit force-pushed the darksky/frontend-copilot-e2e branch 2 times, most recently from 1589f19 to 1f879e2 Compare August 14, 2024 08:22
@darkskygit darkskygit force-pushed the darksky/frontend-copilot-e2e branch 2 times, most recently from 5fed1bc to db5bc91 Compare August 28, 2024 06:28
Brooooooklyn
Brooooooklyn reviewed Oct 28, 2024
View reviewed changes
Copy link
Member

@Brooooooklyn Brooooooklyn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Server Copilot E2E Test takes too much time

@graphite-app Graphite App
Copy link

graphite-app bot commented Oct 29, 2024 •
edited
Loading

Merge activity

@graphite-app graphite-app bot merged commit d718527 into canary Oct 29, 2024
41 checks passed
@graphite-app graphite-app bot deleted the darksky/frontend-copilot-e2e branch October 29, 2024 09:17
Brooooooklyn added a commit that referenced this pull request Oct 31, 2024
@Brooooooklyn
Revert "feat: copilot frontend e2e (#7694)"
c08b02c 
This reverts commit d718527.
darkskygit added a commit that referenced this pull request Oct 31, 2024
@darkskygit
Revert "Revert "feat: copilot frontend e2e (#7694)""
45efa34 
This reverts commit c08b02c.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Brooooooklyn Brooooooklyn Brooooooklyn approved these changes

@EYHN EYHN Awaiting requested review from EYHN

@JimmFly JimmFly Awaiting requested review from JimmFly

@forehalo forehalo Awaiting requested review from forehalo

@CatsJuice CatsJuice Awaiting requested review from CatsJuice

@pengx17 pengx17 Awaiting requested review from pengx17

Assignees
No one assigned
Labels
app:core app:server mod:i18n Related to i18n test Related to test cases
Projects
AFFiNE Project
Status: No status
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@darkskygit @Brooooooklyn @pengx17