Skip to content

[do-not-merge]: tracking diff#11

Closed
Gigas002 wants to merge 61 commits into
mainfrom
test/vk-ref
Closed

[do-not-merge]: tracking diff#11
Gigas002 wants to merge 61 commits into
mainfrom
test/vk-ref

Conversation

@Gigas002
Copy link
Copy Markdown
Owner

@Gigas002 Gigas002 commented Mar 12, 2026

Tracking from testing branch what's done and what's yet to do. After some usage when I'm sure everything works as expected will propose changes to main repo

  1. [PR] freeze option feature: add freeze option waycrate/wayshot#309
  2. [PR] deps upgrades chore: bump dependencies waycrate/wayshot#310
  3. [PR] delay option feature: add delay option waycrate/wayshot#318
  4. [PR] geometry slurp feature: support optional geometry string waycrate/wayshot#333
  5. vulkan module
  6. [PR] egl module + very light refactor (due to moving stuff) refactor: hide egl code behind feature in libwayshot waycrate/wayshot#320
  7. [PR] refactor libwayshot tests refactor: libwayshot tests improvements waycrate/wayshot#330
  8. revert Make Image Buffer Without Clone waycrate/wayshot#290 (or at least find a way to fix it ffs) not needed
  9. add benches not this way I think...
  10. [PR] refactor ci refactor: refactor CI by separating on several files waycrate/wayshot#315
  11. [PR] cargo deny + licensing issues fix: licensing issues waycrate/wayshot#311
  12. [PR] shell completions feature: shell completions waycrate/wayshot#319
  13. improve cli coverage
  14. [PR] filter optimizations fix: use simpler resizing algorithm waycrate/wayshot#329

@Gigas002
Copy link
Copy Markdown
Owner Author

Gigas002 commented Mar 12, 2026

TODO:

  • Consider adding CONTRIBUTING.md with guidelines for new contributors on simple things how to add stuff and why already have one
  • wayshot cli tests
  • is libwayshot's examples for excluded? It should be but better check, in worst case add feature
  • Ci --all-features tests? Probably not possible (especially for vk) but worth taking a look
  • Bench reports in pr response (probably requires some 3rd party bot, so not much hope here) looks too complicated for nothing really useful
  • Node 20 deprecation notice (codecov issue: Node.js 20 deprecation warning caused by actions/github-script dependency codecov/codecov-action#1919)

@Gigas002
Copy link
Copy Markdown
Owner Author

Gigas002 commented Mar 30, 2026

TODO:

  • consider gatekeeping for wlr and dmabuf feats in libwayshot not big win in size/compile time, but kinda annoying refactoring so no point
  • consider gating config/cli feats in wayshot cli no point, since SOME options are specific for cli/config and not replacable which makes app unusable
  • fix dependabot scanning: https://github.com/waycrate/wayshot/network/updates/1300469345
  • notification's config section

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant