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

[bn] Merge dev-bn branch into main branch #3301

Open
wants to merge 314 commits into
base: main
Choose a base branch
from
Open

[bn] Merge dev-bn branch into main branch #3301

wants to merge 314 commits into from

Conversation

sajibAdhi
Copy link
Collaborator

@sajibAdhi sajibAdhi commented Aug 27, 2024

Describe your changes

Merge the dev-bn branch into the main branch.

Related issue number or link (ex: resolves #issue-number)

#410

Checklist before opening this PR (put x in the checkboxes)

  • This PR does not contain plagiarism
    • don’t copy other people’s work unless you are quoting and contributing it to them.
  • I have signed off on all commits
    • signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

mitul3737 and others added 30 commits February 21, 2022 00:44
[bn] Localize Contribute and Contributor Ladder
[bn] changes in file of contributor & contributor ladder
[bn] Localize content\bn\_TEMPLATE.md
Apply CODEOWNERS updates in main to dev-bn
[bn] Localize `content/bn/style-guide/_index.md `
[bn] Updated content/bn/style-guide/_index.md
[bn] Update `content/bn/style-guide/_index.md`
[bn] Updated the _index.md
asem-hamid and others added 14 commits August 20, 2024 19:48
Signed-off-by: Asem Hamid <155321064+asem-hamid@users.noreply.github.com>
Signed-off-by: Asem Hamid <155321064+asem-hamid@users.noreply.github.com>
Signed-off-by: Asem Hamid <155321064+asem-hamid@users.noreply.github.com>
[bn] Update dev-bn branch with main
[bn] fix typo /bn/zero-trust-architecture.md
[bn] Update links microservice > microservice-architecture
…ering

[bn] Fix SRE link in security-chaos-engineering.md
…ecture (#3284)

Signed-off-by: Asem Hamid <155321064+asem-hamid@users.noreply.github.com>
…pe's-observability-section"-link

[bn] Update  cloud native landscape's observability section  link
Signed-off-by: Asem Hamid <155321064+asem-hamid@users.noreply.github.com>
Signed-off-by: Sajib Adhikary <60180521+sajibAdhi@users.noreply.github.com>
Signed-off-by: Asem Hamid <155321064+asem-hamid@users.noreply.github.com>
Co-authored-by: Asem Hamid <155321064+asem-hamid@users.noreply.github.com>
Copy link

netlify bot commented Aug 27, 2024

Deploy Preview for cncfglossary ready!

Name Link
🔨 Latest commit 6ffe849
🔍 Latest deploy log https://app.netlify.com/sites/cncfglossary/deploys/66e1176d2a734c0008538898
😎 Deploy Preview https://deploy-preview-3301--cncfglossary.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@asem-hamid
Copy link
Collaborator

@sajibAdhi , The dev-bn branch is behind a lot of changes, and several files need to be updated.

/hold , until then

@asem-hamid asem-hamid added the hold Wait, please do not proceed this yet label Aug 27, 2024
Copy link

Wait, please do not proceed with this yet.

@jihoon-seo jihoon-seo added the maintainers Use this label if PR requires maintainers to take action label Aug 28, 2024
Copy link

Wait, please do not proceed with this yet.

@jihoon-seo
Copy link
Collaborator

To who merge this PR: please use 'Squash and merge' method when merging.

Signed-off-by: Asem Hamid <155321064+asem-hamid@users.noreply.github.com>
@seokho-son
Copy link
Collaborator

@asem-hamid do you have a plan to unhold this PR?

@sajibAdhi just hope to let you know, this PR has many meaningless commits considering the number of changed files. We recommend localization approvers handle commit history well. (make a localization PR has one commit if possible). Please share this to he team approves. :)

FYI, This PR will be merged by squash merge method.

@asem-hamid
Copy link
Collaborator

@seokho-son No, actually, we are quite behind. We have a small team, so we have to move forward slowly.

Also, we are trying to follow the '1 commit per PR' rule now. However, those meaningless commits are there for historical reasons. In the early days, the team didn't follow the '1 commit per PR' policy strictly, so the dev-bn branch became a junkyard for these commits.

As a result, when we merge the dev-bn branch into the main branch, the maintainer has to use the 'merge by squash' method. Every time we do this, only one commit gets added to the commit history (in the main branch), and the rest of the commits become junk in the dev-bn branch. It has become a repetitive cycle.

To break this cycle, we can take the following step:

After merging this PR (by squash method), we can simply delete the dev-bn branch (if possible) and create a new branch (based on main) with the same name. We'll be careful so that we don't face this issue again.

Please correct me if I’m wrong and share your opinion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hold Wait, please do not proceed this yet lang/bn for Bengali maintainers Use this label if PR requires maintainers to take action
Projects
None yet
Development

Successfully merging this pull request may close these issues.