Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Romadelf
Copy link
Contributor

@Romadelf Romadelf commented Mar 31, 2025

Ignoring files created during configure to be consistent with other files generated from .in templates

During a ./configure run, files are being created from templates using user specific configuration. Most of them are correctly ignored. But somehow, contrib/systemd/[email protected] and contrib/systemd/onedrive.service were left behind and most probably forgotten.

Unless I am mistaken, this commit is a safe minor improvement for the dev side as, with it, these files do not need to be deleted manually anymore before making commits to a forked repo. Needs review.

@github-actions
Copy link

@check-spelling-bot Report

🔴 Please review

See the 📂 files view, the 📜action log, or 📝 job summary for details.

Unrecognized words (41)
bashisms
capitalisation
containerised
customisable
customisation
cuz
finalise
finalised
fullstop
gzipping
initialising
Ioctl
ISPs
kinda
Kubuntu
logfiles
Lubuntu
malware
meh
optimisations
preinstalled
prioritisation
prioritise
prolly
reparse
resynchronisation
sandboxing
segfault
shitload
Slackware
sourced
Sourcing
spamming
thingies
Thu
uninitialised
uninstallation
upgradable
utilisation
websites
Xubuntu
To accept these unrecognized words as correct, you could run the following commands

... in a clone of the [email protected]:Romadelf/onedrive.git repository
on the tweak-.gitignore branch (ℹ️ how do I use this?):

curl -s -S -L 'https://raw.githubusercontent.com/check-spelling/check-spelling/main/apply.pl' |
perl - 'https://github.com/abraunegg/onedrive/actions/runs/14161607913/attempts/1'

@abraunegg
Copy link
Owner

@Romadelf
As per my comment before .......

Please also read: check-spelling/check-spelling#90 (comment)

@Romadelf
Copy link
Contributor Author

Romadelf commented Mar 31, 2025

@abraunegg

Alright, the maintainer has answered here. What do you prefer to do now:

  1. Stall this PR for the "one or two weeks" ETA, serving as a reminder?
  2. Opt for his temporary solve in a separate PR and set up planning to rollback to check-spelling@main when applicable?
  3. Same as 2 but you want me to embed the solve in this PR?
  4. Merge this PR regardless of checks in the meantime, keeping check-spelling@main for ensured consistency?

Note: I'm fine if you hard-delete #3187 as it is no longer relevant and this #3188 is now more concise. I removed the old PR reference in the top comment accordingly.

@abraunegg abraunegg merged commit c851b21 into abraunegg:master Apr 3, 2025
10 of 13 checks passed
@abraunegg abraunegg added this to the v2.5.6 milestone Apr 3, 2025
@Romadelf Romadelf deleted the tweak-.gitignore branch April 4, 2025 10:13
@abraunegg
Copy link
Owner

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

Repository owner locked and limited conversation to collaborators Apr 12, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants