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

Config command #891

Draft
wants to merge 352 commits into
base: development
Choose a base branch
from
Draft

Config command #891

wants to merge 352 commits into from

Conversation

Keyrxng
Copy link
Contributor

@Keyrxng Keyrxng commented Nov 18, 2023

Resolves #888 & #887

Will cover E2E tests over the weekend

Quality Assurance:

@0x4007
chore(supabase): init
bff5d25
@0x4007
chore: generate database types
2a5cb59
@0x4007
fix: jest
ca5a133
@0x4007
fix(database): typings for logging
031d2ff
@0x4007
chore: delete build file
3147c31
@0x4007
chore: make location optional
e344a34
@0x4007
fix: preserve typings in logger of database
f2c9b31
@0x4007
chore(types): simplify node id
3f33c92 
handle logic on database server
@0x4007
chore: ripping everything out from database functions to test and see…
af9098b 
… if this works
@0x4007
chore: update database types
7e0a1ce
@0x4007
refactor: database client
b348b20
@0x4007
feat(tests): user wallet return works
4251c96
@0x4007
test(wallet): get user wallet string only based on user id
264845f
@0x4007
feat: look up wallet set comment url using graphql
295917a
@0x4007
feat: look up url based on node_id of registered wallet
139ff6f
@0x4007
feat: github node type enum for graphql node ids
e6fd88a
@0x4007
feat: github node url works
1e794d8
@0x4007
chore: init rewrite all database functions
e5d3370
@0x4007
test: mock context
79aef02
@0x4007
chore: cleanup
ae1e04d
@0x4007
fix: access but untested
0b557e4
@0x4007
fix: labels access untested
893c265
@0x4007
fix(assign): untested
f050dcd
@0x4007
refactor: break apart functions in payout
680c0c2
@0x4007
refactor: cleanup to reduce build errors
9704ffc
@0x4007
Merge branch 'development' of https://github.com/ubiquity/ubiquibot i…
72fd988 
…nto refactor/general
@0x4007
fix: kebab case
b8a3bfe
@0x4007
Merge branch 'refactor/general' of https://github.com/pavlovcik/ubiqu…
fb08a77 
…ibot into fix/database-functions
@0x4007
fix: wallet test
9e4a6b9
@0x4007
fix: commands test
47d2b52
0x4007 and others added 27 commits November 11, 2023 09:42
@0x4007
refactor: cleanup
d0f62e0
@whilefoo
fix: tests
8897087
@whilefoo
fix: added comment
1100888
@0x4007
refactor: small adjustments
1e1fe26
@0x4007
Merge branch 'refactor/bot-config' of https://github.com/ubiquity-whi…
d057dc6 
…lefoo/ubiquibot into ubiquity-whilefoo-refactor/bot-config
@0x4007
refactor: organize imports
528b8c6
@0x4007
Merge branch 'ubiquity-whilefoo-refactor/bot-config' into feat/contri…
54a7066 
…butor-incentives-total-scoring
@0x4007
fix: build errors
a0e488e
@0x4007
fix(config): token limit moved to misc
53428a5
@0x4007
fix(config): loader still not stable
e1be6a8
@0x4007
refactor: cleanup
d8fd54c
@whilefoo
feat: remove zod
232cecd
@whilefoo
fix: merge before validation/defaults
4cc7866 
reference: 0x4007#38 (comment)
@whilefoo
feat: custom merge for arrays and change commands to disabledCommands
ca1d827 
also changed time and priority labels from array of objects to array of strings
@0x4007
Merge pull request 0x4007#45 from ubiquity-whilefoo/fix-config
62cb0c6 
Fix config
@0x4007
feat: logs do not render in console or comment if filtered out based …
4ccce56 
…on log level
@0x4007
feat: logs do not render in comment if filtered out and if bubbled to…
2949f7e 
… top via return etc
@0x4007
fix: refactored 'ubiquibot-config.yml' and redesigned logger statements
3806a89
@0x4007
feat: comment on commit
6253435
@0x4007
feat: improved comment posting and structured metadata util
5dfc27e
@wannacfuture
feat: duplication fix
af772be
@wannacfuture
feat: minor fix
ad8e068
@wannacfuture
feat: minor fix
880281c
@0x4007
Merge pull request 0x4007#48 from pavlovcik/feat/contributor-incentiv…
d00946c 
…es-total-scoring-duplicate-fix
@Keyrxng
feat: command init
2c36717
@Keyrxng
feat: basic help
d2132bc
@Keyrxng
feat: infer, inject, push
a2ec9fa
@netlify Netlify
Copy link

netlify bot commented Nov 18, 2023 •
edited

❌ Deploy Preview for ubiquibot-staging failed.

Name Link
🔨 Latest commit a2ec9fa
🔍 Latest deploy log https://app.netlify.com/sites/ubiquibot-staging/deploys/6558b6a1e9e20c000896a100

@Keyrxng
Copy link
Contributor Author

Keyrxng commented Nov 18, 2023 •
edited

@pavlovcik Forgive the noobness here but is this the best way to merge from your current refactor branch? Seeing all of those commits gives me the fear but I expect once merged it shouldn't make a difference or is this a disaster? 🤣

@0x4007
Copy link
Member

0x4007 commented Nov 20, 2023 •
edited

I don't understand your question but technically speaking we don't have tasks open for contributors on this repository because of the anticipated complexities with merging. If you're particularly motivated to try even with this relatively adversarial merge environment (refactor is the top priority) then I would recommend you base your branch off of the latest, which actually is feat/contributor-incentives-total-scoring

For context, I'll try risky changes on a sub-branch, get them pretty stable, then merge to my main branch. In the context of the refactor, my "main branch" is refactor/general

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@Keyrxng @0x4007 @whilefoo @wannacfuture