Jump to content

All Activity

This stream auto-updates     

  1. Past hour
  2. Hey guys, I've had more of a chance to play around with the code review system now. Love it! I think you guys have done a really good job making it so user-friendly. Love being able to quickly see which questions / change requests are still pending and double-clicking to the correct diff. Great job guys. A few minor bugs I've noticed: The "Close change requested in review" dropdown isn't showing up for me in the Pending Changes view, even when there are active change requests (latest Windows version)? Can still close change requests by typing the comment manually though... The "Review entire branch" mode doesn't refresh when a new changeset is created. The bottom review pane and the "review changeset by changeset" mode do though šŸ‘ The Code Review view highlights "rework required" code reviews in green (I'm guessing this is a new status and someone forgot to change the if-statement for the highlighting šŸ˜œ) Some minor improvements that would make the experience even better: Automatically change code review status in some cases, eg: Change to "rework required" if a new change request is made. Change to "pending review" if a new changeset/question is created after already marked as "Reviewed". Since Branch Explorer is the main view, it would be great if there was better integration of the code review system with this view, eg: "Display option" to show some kind of visual indicator of the code review status (eg, not created, pending, etc) and number of open questions/changes A way to easily open an existing code review from Branch Explorer, eg either from right-click menu or by double-clicking (perhaps if a pending code review exists for a branch, launch that when double-clicking the branch rather than the branch diff) Prevent changes to code review comments once a branch has already been merged to main. (But unlock the code review if a new changeset is created on the branch after merge - not the recommended approach, but in case anyone does this.) Only semi-related but noticed this when playing around with the new attribute default values (thank you!!! and can finally edit attribute comments šŸ¤£). Colours automatically update when applying new conditional formatting rules, but not if just changing attributes (have to manually refresh the view). Can you make it so changing attributes automatically triggers a refresh? Lastly, is it safe to start using the new code review system for actual repos? In your latest blog posts, it says it's still a prototype and so comments won't be stored. But they seem to be stored just fine for me? Or is there a possibility you'll introduce some breaking back-end changes down the track? Great work guys, loving all the recent useability changes šŸ˜˜ David
  3. Yesterday
  4. I think you are missing the full story by not running the entire test. My machine reached 45k imported changesets after 2.5 hours - it is around the 50k-60k mark that I begin to see real performance problems. Here are some tools for profiling the situation: https://github.com/kalmalyzer/profile-gitsync in case you are interested. I have let the sync step run for approx 12 hours now. I am syncing to a local server. I interrupted it after it had imported 72k changesets out of 140k. Here is where the time has been spent: Stage Duration (seconds) Compressing objects 6.402 Downloading 219.25 Processing objects 2823.818 Importing 42205.704 This graph shows the average time taken (in seconds) for importing each changeset. The light-blue line represents individual values, the dark-blue line is a trend line, formed with moving average, over the 10 nearest samples: What this means is, importing the remaining 68k changesets will take a long time. - Even if the trend would suddenly stop, and land at 2 seconds/changeset, it would take another 37 hours to import the remaining changesets. - More likely, the trend continues to rise with 1 second per 10k changesets, and it will take 4 days and 20 hours to import the remaining changesets. I will disable Windows Update for a month and continue running this on my machine, just to see what the trend looks like.
  5. Last week
  6. Hi, Is there any way to configure how big a file needs to be for Plastic to deem it too big for automatic diff when viewing pending changes? Or maybe a way to tell plastic to always automatically diff a particular file or even all files? Having to always manually click the "Calculate differences" button for a particular file (Unity scene file in this case) wastes time and interrupts flow. At the very least the current threshold seems flawed. When actually doing the diff takes about a third of a second and clicking the button takes two seconds then that's a bad experience. Thanks, Mikael
  7. Koop rijbewijs Nederland, ID-kaarten, Visa-paspoorten (((Globedocumentshotmail.com ))) contact: +44 7424 163167 Wij zijn de beste producenten van echte en valse documenten van hoge kwaliteit, waaronder paspoorten, rijbewijzen, identiteitskaarten, Postzegels, visa enz. Met miljoenen van onze documenten die over de hele wereld circuleren. Wij bieden alleen originele hoge kwaliteit Echte / nep-paspoorten, rijbewijzen, identiteitskaarten, visa, geboortecertificaten enz. En andere producten voor een aantal landen zoals: VS, Australiƫ, Belgiƫ, Braziliƫ, Canada, Finland, Frankrijk, Duitsland, Israƫl, Mexico, Nederland, Zuid-Afrika, Spanje, Verenigd Koninkrijk, Nederland, Pakistan, Rusland, Spanje, Oostenrijk, Nieuw-Zeeland, Crotia, Roemeniƫ, Italiƫ enz. -ID's Scan-ja ... -HOLOGRAMS: IDENTIEK -BARCODES: IDS-SCAN -UV: JA IDS MET SNELLE VERZENDING - E-MAILONDERSTEUNING CONTACT Algemene ondersteuning ::::::::::: ((((Globedocumentshotmail.com ))) Whats App Tel / :::::::::::::::::::: +44 7424 163167 ID's voor meer dan 50's rijbewijskaarten en staat ID kaarten beschikbaar. Inclusief staatshologram en magneetstrip of barcode op keerzijde. 2 bussiness day verzending bestelling (2,95 verzending en handling). De BESTE authentieke ID's op internet. Dezelfde kwaliteit, hoge resolutie die het Department of Motor Vechiles gebruikt. Stuur huidige .JPG of .GIF van huidige licentie met afbeelding en wijzigingen in NAAM, DOB, LICENTIE #, BEPERKINGEN, enz. wij produceren de beste ID online we verkopen UK / EU ID, Canadese ID, Australische ID en ID's voor veel andere landen van paspoorten voor rijbewijzen tot bankafschriften hebben we uw identificatiebehoeften gedekt. Onze ID's omvatten alle beveiligingsfuncties zoals echte hologrammen, ultraviolette watermerken, diepdruk, speciaal papier, fluorescerende kleurstoffen, RFID-chips, barcodes overeenkomt met uw gegevens en meer. onze ID's zijn identiek aan het echte werk, geen enkele andere site biedt u deze kwaliteit wij zijn de besten. nieuwe identiteit beschermt uw privacy en neemt uw vrijheid terug. trefwoorden Canada Kaarten Verenigde Staten kaarten Studentenkaarten Internationale kaarten PrivƩ kaarten Adoptie Certificaten Doopcertificaten Geboorte certificaat Overlijdensakte Echtscheidingscertificaten Huwelijkscertificaten Aangepaste certificaten Middelbare School Diploma's G.E.D. diploma's Home School Diploma's College graden Universitaire diploma's Trade Skill Certificates Sociale zekerheid Valideer SSN-nummer Rijbewijs zoeken Spion producten Voice Changers Luistertoestellen Onzichtbare inkt Aanvraag DMV-record Achtergrondcontrole Iedereen onderzoeken koop echte Nederlandse (Nederland / Holland) rijbewijzen, koop echte Duitse (Duitsland) rijbewijzen, koop echte VK (Verenigd Koninkrijk) rijbewijzen, koop echte rijbewijzen in het VK (Belgiƫ) koop echte diplomatieke paspoorten, CONTACT Algemene ondersteuning ::::::::::: ((((Globedocumentshotmail.com ))) Whats App Tel / :::::::::::::::::::: +44 7424 163167
  8. Wij zijn de beste producenten van echte en valse documenten van hoge kwaliteit, waaronder paspoorten, rijbewijzen, identiteitskaarten, Postzegels, visa enz. Met miljoenen van onze documenten die over de hele wereld circuleren. Wij bieden alleen originele hoge kwaliteit Echte / nep-paspoorten, rijbewijzen, identiteitskaarten, visa, geboortecertificaten enz. En andere producten voor een aantal landen zoals: VS, Australiƫ, Belgiƫ, Braziliƫ, Canada, Finland, Frankrijk, Duitsland, Israƫl, Mexico, Nederland, Zuid-Afrika, Spanje, Verenigd Koninkrijk, Nederland, Pakistan, Rusland, Spanje, Oostenrijk, Nieuw-Zeeland, Crotia, Roemeniƫ, Italiƫ enz. -ID's Scan-ja ... -HOLOGRAMS: IDENTIEK -BARCODES: IDS-SCAN -UV: JA IDS MET SNELLE VERZENDING - E-MAILONDERSTEUNING CONTACT Algemene ondersteuning ::::::::::: ((((Globedocumentshotmail.com ))) Whats App Tel / :::::::::::::::::::: +44 7424 163167 ID's voor meer dan 50's rijbewijskaarten en staat ID kaarten beschikbaar. Inclusief staatshologram en magneetstrip of barcode op keerzijde. 2 bussiness day verzending bestelling (2,95 verzending en handling). De BESTE authentieke ID's op internet. Dezelfde kwaliteit, hoge resolutie die het Department of Motor Vechiles gebruikt. Stuur huidige .JPG of .GIF van huidige licentie met afbeelding en wijzigingen in NAAM, DOB, LICENTIE #, BEPERKINGEN, enz. wij produceren de beste ID online we verkopen UK / EU ID, Canadese ID, Australische ID en ID's voor veel andere landen van paspoorten voor rijbewijzen tot bankafschriften hebben we uw identificatiebehoeften gedekt. Onze ID's omvatten alle beveiligingsfuncties zoals echte hologrammen, ultraviolette watermerken, diepdruk, speciaal papier, fluorescerende kleurstoffen, RFID-chips, barcodes overeenkomt met uw gegevens en meer. onze ID's zijn identiek aan het echte werk, geen enkele andere site biedt u deze kwaliteit wij zijn de besten. nieuwe identiteit beschermt uw privacy en neemt uw vrijheid terug. trefwoorden Canada Kaarten Verenigde Staten kaarten Studentenkaarten Internationale kaarten PrivƩ kaarten Adoptie Certificaten Doopcertificaten Geboorte certificaat Overlijdensakte Echtscheidingscertificaten Huwelijkscertificaten Aangepaste certificaten Middelbare School Diploma's G.E.D. diploma's Home School Diploma's College graden Universitaire diploma's Trade Skill Certificates Sociale zekerheid Valideer SSN-nummer Rijbewijs zoeken Spion producten Voice Changers Luistertoestellen Onzichtbare inkt Aanvraag DMV-record Achtergrondcontrole Iedereen onderzoeken koop echte Nederlandse (Nederland / Holland) rijbewijzen, koop echte Duitse (Duitsland) rijbewijzen, koop echte VK (Verenigd Koninkrijk) rijbewijzen, koop echte rijbewijzen in het VK (Belgiƫ) koop echte diplomatieke paspoorten, CONTACT Algemene ondersteuning ::::::::::: ((((Globedocumentshotmail.com ))) Whats App Tel / :::::::::::::::::::: +44 7424 163167
  9. Hi, I started the same clone operation this morning on my local PC. Around 7 hours running: Importing... \ 45784/138997 I'm using the "cm sync" (next time please also use the "cm" so we can get more feedback on the operation). I'm using the office network and my SSD drive. It doesn't seem to be as fast as Manu reported but it should be finishing in less than 24 hours (faster than your test). Anyway, the initial operation is what should take a longer time (this is a big repo). But after that, the sync for the new commits should be very fast. Regards, Carlos.
  10. Hi @psantosl and team, Super excited to see all the improvements you've been making lately, especially the new conditional formatting of changesets/labels (yay!!! šŸ˜˜) and of course the new code review system! My time's been taken up with some other things these last few months, and I haven't had much time to play around with Plastic unfortunately (or coding in general for that matter), but I've been keeping an eye on the regular release notes and can't wait to get back into things soon. I had been keeping a small list of thoughts and suggestions when I was playing around with different workflows, and more recently when reading up on the new code review system, but it looks like you've beaten me to a lot of those already (like conditional formatting of labels/changesets, and hiding non-question and non-change code review comments to give more screen space)! So since you guys are moving much faster than me, I thought I'd share some of the thoughts that I've had for some time now, in case any of it is helpful. I've been reading the release notes diligently, so I don't think any of these have been implemented already, but apologies (and great job) if you already have! Updating changeset comments in DVCS scenario I have an unfortunate habit in that I often check in something and only realise I made a mistake a few changesets later (I'm sure I'm not the only one šŸ˜œ). Colour coding changesets is great for warning about these "bad" changesets (thanks!!), but another thing I like to do is edit the changeset comment to identify the error and which changeset it was fixed in, eg I used to add something like this to the end of the comment: ERROR: forgot to update comment to reflect changed code. [Fixed in cs:1234] In many ways, I think this is very similar to the "review scripts" feature you guys are implementing for the code review system (which I really like by the way), eg [refactor: Check how I extracted the method and changed order: BranchExplorerLegendItems.cs : left:167-174@cset:170, right:180-187]. The issue I've come across is the way that edited comments are handled in DVCS scenarios. It's fine when working centralised, but with push/pull model it doesn't check whether comments have been changed - presumably this is for performance reasons as changeset history is not stored so there's no easy way to tell if a comment has changed? I imagine once "review scripts" become a thing, more people might want to tidy-up changeset comments ahead of reviews (rather than having to get it right the first time - which I often don't, especially if I've had to change my approach because there was an issue I didn't anticipate). I have changed to centralised model because of this and that's ok for me, but others might prefer to stick with distributed model. Ideally, the ability to track and diff comment history would be neat - perhaps using a simplified revision tree for comments but without needing to worry about branching/merging etc. I think possibly that might be more performant than checking all changesets in br:main/ for comment timestamp (yikes!), but understand this would probably require non-trivial architectural changes... Review script cset numbers in DVCS scenario On a related note, an issue I anticipate with specifying csets in "review script" comments is that cset numbers can vary between replicated servers. Not sure best way to fix this since specifying GUID would be long and ugly... Something that I was doing before (and which might help here), is to basically label every meaningful changeset, even on task branches. (The reason I was doing this is because I was trying out this approach where I'd track, inside a text file, a list of things that needed to be done/checked/tested before the task branch was allowed to be merged into main. Changesets with actual changes would be labelled, but changesets that just updated the checklist because I'd thought of something new would not be labelled. By the way, I'd then delete the text file when merging so main branch wouldn't get polluted with all these checklists - I'd have to delete the file via File Explorer and the Pending Changes view would pop up an error, but the merge would work on the second try.) Reason I mention this is because referring to labels neatly gets around the inconsistent cset number issue. Another benefit of this approach is that label names show up in Branch Explorer, so it's easy to see at a glance which changeset is being referred to in a comment, etc. The convention I was using is "M-1" for changeset 1 on main and "2-1" for changeset 1 on task branch 2 - which is easy to understand and reference. (The new conditional formatting for labels is great for this by the way, as it means you can easily distinguish between these "regular" labels and "release" labels šŸ‘) Related GUI issue: label names on child branches often get hidden by the task branch name in Branch Explorer. Perhaps move the label name so it appears below the changeset/branch (rather than above, which is where the branch name goes)? Preserve line break formatting in multi-line comments Something I already have a lot of, and think will be used even more once "review scripts" are standard (eg, you give the following example on your blog): [file: Simple.cs] Contains the redesign of the calculation. [file: Calc.cs] Contains the code extracted from BaseCommandsImpl. [group: Just method renamed: Bar.cs, Foo.cs, BaseCommandsImpl.cs] Currently (or at least last time I tried), line break formatting is discarded in the popup when mousing over changesets in Branch Explorer, etc. It is also very hard to read multi-line comments at the top of diff windows (although I understand that is probably done to preserve screen real estate). It would be great if both the formatting and "clickability" of review scripts is preserved in all places where comments are displayed (not just the code review window). Server-side trigger for "on comment update" Related to suggestion 1 above - would be helpful to be able to create a trigger that automatically applies an attribute if a certain phrase is added (eg, [error: ā€¦]). Less important now that I've recently discovered you can create custom "External Tools" to apply attributes (http://blog.plasticscm.com/2019/06/archive-branches-in-plastic-scm.html), but before it was a real pain having to navigate to the "Attributes" side panel each time. As an aside, why not combine the "Attributes" panel with the panel that shows/edits the comment field (which is the panel I usually have open so I can see comments easily)? New trigger exit code that warns user and gives option to proceed or cancel Came up with this one with @manu a while back. For client-side "before checkin" trigger, I was able to create a simple powershell script that pops up a window and gives user option to proceed or cancel (returns exit code 0 or 1 depending on which button user clicked). This is useful for warning about some things that are usually bad (eg, trailing whitespaces, weird characters in file names, etc) but which might not be appropriate to enforce in all cases - hence warning the user and giving them the option to proceed or cancel. The above is the most important use case, but it would also be helpful for some server-side triggers (eg, checking comments when something is created/updated). Perhaps returning a new exit code 2 or something could make the Plastic client pop up a window that lets you proceed or cancel the relevant action? Add "labels" column in Changeset View I know we Plastic users don't use Changeset View much, but I still find it useful occasionally when I want to see a quick summary of recent changes on main branch (probably a leftover habit from Git šŸ˜œ). Being able to see the labels would be helpful in this situation. On a related note, either adding an easy "show main only" toggle button or making the Changeset View remember the filter would be helpful (last time I used Plastic, it wasn't remembering my filter so I had to keep manually typing it out each time). Thanks again guys for continuing to make Plastic better and better. Love what you guys have been doing! David
  11. As an update for the forum post (we have been also chatting via support email): As you probably saw in our blog, we are currently redoing the entire code review system to come up with something much more powerful, focusing on smoothing the interactions between reviewers and authors.Right now the first phase for Windows has been released, but more work is yet to be done.We'll start on the macOS side of things real soon, and our goal is to reach the current Windows feature parity before the end of 2019.Regards, Carlos.
  12. Hey Carlos, Am I understanding correctly that there is no solution for code reviews on Cloud Edition with non-windows machines (Specifically, Mac)? šŸ˜• Regards, Ivan
  13. Hi, I'm afraid the redesigned webUI (including the code review support) is only available for Plastic hosted servers http://blog.plasticscm.com/2018/09/webui-web-interface-for-plastic.html Regards, Carlos.
  14. Hi, I'm afraid it's not currently configurable but it's shouldn't be very difficult to enable it https://plasticscm.uservoice.com Regards, Carlos.
  15. Hey guys, I've been setting up Plastic Cloud Edition, but have been unable to get Code Reviews to work on our Mac machines. Upon contacting support, I was told that these users could use the WebUI, but I've been unable to find anything about how to do so. Thus, my question: How do I access the WebUI for a Cloud only repository? I might need more info once I get that far, but I do not know that yet. Kind Regards, Ivan
  16. Hi, I'm afraid it's not possible. You will need to re-create the Xlink in order to change the writable flag. Regards, Carlos.
  17. Hi, A branch rename is not pushed during a replica operation. If you locally renamed a branch and also want to propagate this change to cloud, you will also need to manually rename the branch in the cloud repo. Regards, Carlos.
  18. Hi, It will only appear for files that are not under source control. You can only ignore private files. For controlled files, you can only add to "cloaked"/"hidden changes" list. Regards, Carlos.
  19. Is there any way to rename branches on the cloud? Renaming them in the client only seems to affect the local repo, and syncing to cloud doesn't seem to push name changes. The original name persists when setting up a new workstation.
  20. Ah, so my client actually doesn't have that option. I'm on 8.0.16.3594 Cloud Edition.
  21. Me and my fellow artists love the fact that users are colored in Gluon changesets. Is it possible to change these colors somewhere? Some of us are not happy with the color the system(?) has assigned to us.
  22. Hi there! Do we have a opportunity to switch back and forth from xlink to wxlink for a quick changes? Thx.
  23. Hi, Are you using Plastic GUI or Gluon? If you are using Plastic GUI and there are newer changes in your working branch, you will need to update your workspace first to the last changeset of the branch (involving a merge if there are file conflicts) and then you will be able to commit your local changes. Regards, Carlos.
  24. Earlier
  25. Hi, When you right-click and select add to ignore list, you are actually adding a new rule in the "ignore.conf" file. When you add the rules, you can select to apply it only for the local workspace or not. (you can have both a local "ignore.conf" and a global "ignore.conf") Best regards, Carlos.
  26. Hi, just wondering what's the rationale behind only having right-click menu shortcuts for adding to cloaked and hidden changes but not to the general ignore.conf. EDIT: Sorry, put this in the wrong subsection, should be in PlasticSCM/General.
  27. Hi Carlos! Still wondering if there's a particular solution to this problem. I haven't had time to dig deeper into it. Even if there isn't an easy/established way to query a changeset's attributes, let me know
  28. Hi, Plastic automatically does it if you run an update operation that requires downloading a file and you already have a local private file with the same name. Plastic doesn't overwrite your local private files but renames them. Regards, Carlos.
  1. Load more activity
×
×
  • Create New...